From: whitequark <whitequark@whitequark.org>
Date: Wed, 21 Aug 2019 21:32:12 +0000 (+0000)
Subject: vendor.lattice_ice40: remove `--placer heap` default option.
X-Git-Tag: locally_working~9
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=5889e626680f9e7f95ba8968d5247b92380720a4;p=nmigen.git

vendor.lattice_ice40: remove `--placer heap` default option.

It is not the place of nMigen to decide on this default, since both
SA and HeAP have valid uses that are not covered by the other.
---

diff --git a/nmigen/vendor/lattice_ice40.py b/nmigen/vendor/lattice_ice40.py
index 45aaac4..5c1880b 100644
--- a/nmigen/vendor/lattice_ice40.py
+++ b/nmigen/vendor/lattice_ice40.py
@@ -24,8 +24,7 @@ class LatticeICE40Platform(TemplatedPlatform):
         * ``script_after_read``: inserts commands after ``read_ilang`` in Yosys script.
         * ``script_after_synth``: inserts commands after ``synth_ice40`` in Yosys script.
         * ``yosys_opts``: adds extra options for ``yosys``.
-        * ``nextpnr_opts``: adds extra and overrides default options for ``nextpnr-ice40``;
-          default options: ``--placer heap``.
+        * ``nextpnr_opts``: adds extra options for ``nextpnr-ice40``.
 
     Build products:
         * ``{{name}}.rpt``: Yosys log.
@@ -107,7 +106,7 @@ class LatticeICE40Platform(TemplatedPlatform):
         r"""
         {{get_tool("nextpnr-ice40")}}
             {{quiet("--quiet")}}
-            {{get_override("nextpnr_opts")|default(["--placer","heap"])|options}}
+            {{get_override("nextpnr_opts")|options}}
             --log {{name}}.tim
             {{platform._nextpnr_device_options[platform.device]}}
             --package