Use internal `run_command()` API instead of `popen()`.

Co-Authored-By: Claire Wolf <claire@symbioticeda.com>

Clean up manual casting.

Co-Authored-By: David Shah <dave@ds0.me>

Remove unimplemented `-timeout` option.

Implement the `-assume-outputs`, `-sat`, and -unsat` options for the `qbfsat` command.

Add NDEBUG guards to `qbfsat` assertions.

Implement `-specialize-from-file` option for the `qbfsat` command.

Implement `-write-solution` option for the `qbfsat` command.

Clean up `passes/sat/qbfsat.cc`.

Updated `yosys-smtbmc` to optionally dump raw bit strings, and fixed hole value recovery using that mode.

Hole value recovery and specialization implementation for `qbfsat` command.

Barebones implementation of `qbfsat` command.

Initial skeleton for `qbfsat` command.

Rename `-duplicate` to `-push-copy`.

Co-Authored-By: whitequark <whitequark@whitequark.org>

Add `-duplicate` option to the `design` command.

Merge pull request #1783 from boqwxp/astcc_cleanup

Clean up pseudo-private member usage in `frontends/ast/ast.cc`.

Merge pull request #1835 from boqwxp/cleanup_sat_expose

Clean up pseudo-private member usage in `passes/sat/expose.cc`.

Merge pull request #1832 from boqwxp/cleanup_passes_cmds_design

Clean up pseudo-private member usage in `passes/cmds/design.cc`.

Merge pull request #1786 from boqwxp/hierarchycc_cleanup

Clean up pseudo-private member usage in `passes/hierarchy/hierarchy.cc`.

Add explanatory comment about inefficient wire removal and remove superfluous call to `fixup_ports()`.

Co-Authored-By: Eddie Hung <eddie@fpgeh.com>

Merge pull request #1831 from boqwxp/cleanup_sat_eval

Clean up pseudo-private member usage in `passes/sat/eval.cc`.

Merge pull request #1833 from boqwxp/cleanup_sat_freduce

Clean up pseudo-private member usage in `passes/sat/freduce.cc`.

Remove unused function parameter.

Simplify iterating over selected modules or cells.

Co-Authored-By: N. Engelhardt <nak@symbioticeda.com>

Replace `RTLIL::id2cstr()` with `log_id()`.

Co-Authored-By: Eddie Hung <eddie@fpgeh.com>

Fix double deletion in `passes/hierarchy/hierarchy.cc`.

Co-Authored-By: Eddie Hung <eddie@fpgeh.com>

Further clean up `passes/sat/eval.cc`.

Co-Authored-By: Eddie Hung <eddie@fpgeh.com>

Further clean up `passes/sat/freduce.cc`.

Co-Authored-By: Eddie Hung <eddie@fpgeh.com>

Clean up more in `passes/sat/expose.cc`.

Co-Authored-By: N. Engelhardt <nak@symbioticeda.com>

Merge pull request #1811 from PeterCrozier/typedef_scope

Support module/package/interface/block scope for typedef names.

Merge pull request #1778 from rswarbrick/sv-defines

Add support for SystemVerilog-style `define to Verilog frontend

Explicit include of csignal

windows - there are no stopping signals

Clean up pseudo-private member usage in `passes/sat/expose.cc`.

Clean up pseudo-private member usage in `passes/sat/freduce.cc`.

Clean up pseudo-private member usage in `passes/cmds/design.cc`.

Clean up pseudo-private member usage in `passes/sat/eval.cc`.

Merge pull request #1607 from whitequark/simplify-simplify-meminit

ast: avoid intermediate wires/assigns when lowering to AST_MEMINIT

Inline productions to follow house style.

Add support for SystemVerilog-style `define to Verilog frontend

This patch should support things like

  `define foo(a, b = 3, c)   a+b+c

  `foo(1, ,2)

which will evaluate to 1+3+2. It also spots mistakes like

  `foo(1)

(the 3rd argument doesn't have a default value, so a call site is
required to set it).

Most of the patch is a simple parser for the format in preproc.cc, but
I've also taken the opportunity to wrap up the "name -> definition"
map in a type, rather than use multiple std::map's.

Since this type needs to be visible to code that touches defines, I've
pulled it (and the frontend_verilog_preproc declaration) out into a
new file at frontends/verilog/preproc.h and included that where
necessary.

Finally, the patch adds a few tests in tests/various to check that we
are parsing everything correctly.

Merge pull request #1815 from boqwxp/fix-ef-optimize

Fix solver output parsing for exists-forall optimization

Revert over-aggressive change to a more modest cleanup.

Do not change solver output parsing for non-exists-forall problems.

Merge pull request #1806 from YosysHQ/mwk/techmap-replace-fix

techmap: Fix cell names with _TECHMAP_REPLACE_.*

Skip reading stdout from the solver that if it isn't a line reading only "sat", "unsat", or "unknown".

Revert part of 0fda8308 from #1746 that broke other smtbmc flows

Signed-off-by: Claire Wolf <claire@symbioticeda.com>

Error duplicate declarations of a typedef name in the same scope.

Support module/package/interface/block scope for typedef names.

Merge pull request #1763 from boqwxp/issue1762

Closes #1762. Adds warnings for `select` arguments not matching any object and for `add` command when no modules selected

Do not warn on empty selection with prefixed `arg_memb`.

Co-Authored-By: N. Engelhardt <nak@symbioticeda.com>

Suppress warnings for empty `select` arguments when `-count` or `-assert-*` options are set.

Add tests for `select` command warnings.

Merge pull request #1803 from Grazfather/typedef

Support standard typedef grammar (Fixed)

Merge pull request #1805 from hofstee/master

fix typo in `write_smt2` help

techmap: Fix cell names with _TECHMAP_REPLACE_.*

Fixes #1804.

Merge pull request #1785 from boqwxp/mitercc_cleanup

Clean up pseudo-private member usage in `passes/sat/miter.cc`.

fix typo in `write_smt2` help

Warn on empty selection for `add` command.

Simplify was not being called for packages.  Broke typedef enums.

Build pkg_user_types before parsing in case of changes in the design.

Clear pkg_user_types if no packages following a 'design -reset-vlog'.

Revert typedef tests to standard grammar.

Parser changes to support typedef.

iopadmap: Attempt to give new wires/cells meaningful names

Merge pull request #1794 from YosysHQ/dave/mince-abc9-fix

ice40: Map unmapped 'mince' DFFs to gate level

Merge pull request #1795 from smunaut/fix_abc9_spram

ice40: Fix typos in SPRAM ABC9 timing specs

ice40: Fix typos in SPRAM ABC9 timing specs

Signed-off-by: Sylvain Munaut <tnt@246tNt.com>

Add test for abc9+mince issue

Signed-off-by: David Shah <dave@ds0.me>

ice40: Map unmapped 'mince' DFFs to gate level

Signed-off-by: David Shah <dave@ds0.me>

xilinx: Mark IOBUFDS.IOB as external pad

Merge pull request #1788 from YosysHQ/eddie/fix_ndebug

Fix NDEBUG warnings

fsm_extract: Initialize celltypes with full design.

Fixes #1781.

Merge pull request #1787 from YosysHQ/mmicko/lexer_deps

Add dependency to verilog_lexer.cc

Add one mode dependency

Fix NDEBUG warnings

Merge pull request #1774 from boqwxp/exec

Add `exec` command to allow running shell commands from inside Yosys scripts

Merge pull request #1775 from huaixv/asserts_locations

Add precise locations for asserts

Clean up pseudo-private member usage in `frontends/ast/ast.cc`.

Clean up pseudo-private member usage in `passes/sat/miter.cc`.

Clean up pseudo-private member usage in `passes/hierarchy/hierarchy.cc`.

Add precise locations for asserts

Merge pull request #1780 from YosysHQ/fix-test-bash

Fix make test on macOS: add bash to brewfile & fix argument order

fix argument order for macOS compatibility

add bash to brewfile

Default bash on macOS is version 3, which does not support `define -A`, now used in `tests/arch/run-test.sh`. Use brew to install newer bash.

Update copyright and license header.

I hereby assign to Claire Wolf the copyright for all work I did on `passes/cmds/exec.cc`.
In the event that this copyright assignment is not legally valid, I offer this work under the ISC license.

Clean up `exec` code according to review.

Co-Authored-By: Miodrag Milanović <mmicko@gmail.com>

Add N:* to select language, fix some old code

Signed-off-by: Claire Wolf <claire@symbioticeda.com>

Merge pull request #1769 from boqwxp/select_cleanup

Clean up code style and pseudo-private member usage in `passes/cmds/select.cc`

Merge pull request #1776 from boqwxp/precise_locations

Add AST node source location information in a couple more parser rules.

Add AST node source location information in a couple more parser rules.

Further clean up `passes/cmds/select.cc`.

Co-Authored-By: Eddie Hung <eddie@fpgeh.com>

Cleanup code style and pseudo-private member usage in `passes/cmds/select.cc`.

Update Copyright

Signed-off-by: Claire Wolf <claire@symbioticeda.com>

Merge pull request #1771 from waldyrious/license-update

License: bump year and add title

Merge pull request #1768 from boqwxp/smt2_cleanup

Clean up pseudo-private member usage in `backends/smt2/smt2.cc`.

Merge pull request #1746 from boqwxp/optimization

Add support for optimizing exists-forall problems.

Add test for `exec` command.

Allow specifying multiple regexes to match in `exec` command output, and also to specify regexes that must _not_ match.

Add `exec` command to run shell commands.

Merge pull request #1773 from smunaut/fix_spram_model

ice40: Fix SPRAM model to keep data stable if chipselect is low

ice40: Fix SPRAM model to keep data stable if chipselect is low

According to the official simulation model, and also cross-checked
on real hardware, the data output of the SPRAM when chipselect is
low is kept stable. It doesn't go undefined.

Signed-off-by: Sylvain Munaut <tnt@246tNt.com>