aboutsummaryrefslogtreecommitdiffstats
path: root/passes
Commit message (Collapse)AuthorAgeFilesLines
...
| * | opt_dff: fix sequence point copy paste bugAustin Seipp2022-01-041-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Newer GCCs emit the following warning for opt_dff: passes/opt/opt_dff.cc:560:17: warning: operation on ‘ff.Yosys::FfData::has_clk’ may be undefined [-Wsequence-point] 560 | ff.has_clk = ff.has_ce = ff.has_clk = false; | ~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ Which is correct: the order of whether the read or write of has_clk occurs first is undefined since there is no sequence point between them. This is almost certainly just a typo/copy paste error and objectively wrong, so just fix it. Signed-off-by: Austin Seipp <aseipp@pobox.com>
| * | memory_share: Fix SAT-based sharing for wide ports.Marcelina Kościelnicka2021-12-201-1/+3
| | | | | | | | | | | | Fixes #3117.
| * | bugpoint: avoid infinite loop between -connections and -wires.Catherine2021-12-151-1/+1
| | | | | | | | | | | | Fixes #3113.
| * | Add clean_zerowidth pass, use it for Verilog output.Marcelina Kościelnicka2021-12-122-1/+212
| | | | | | | | | | | | | | | | | | | | | This should remove instances of zero-width sigspecs in the netlist, avoiding problems in the Verilog backend with emitting them. See #3103.
| * | opt_mem_priority: Fix non-ascii char in help message.Marcelina Kościelnicka2021-12-091-1/+1
| | | | | | | | | | | | This is a fixed version of #3072.
| * | sta: very crude static timing analysis passLofty2021-11-253-30/+341
| | | | | | | | | | | | Co-authored-by: Eddie Hung <eddie@fpgeh.com>
| * | show: Fix wire bit indexing.Marcelina Kościelnicka2021-11-121-3/+16
| | | | | | | | | | | | Fixes #3078.
| * | Merge pull request #3077 from YosysHQ/claire/genlibClaire Xen2021-11-101-21/+40
| |\ \ | | | | | | | | Add genlib support to ABC command
| | * | Spelling fix in abc.ccClaire Xen2021-11-101-1/+1
| | | |
| | * | Add genlib support to ABC commandClaire Xenia Wolf2021-11-101-21/+40
| | | | | | | | | | | | | | | | Signed-off-by: Claire Xenia Wolf <claire@clairexen.net>
| * | | iopadmap: Fix ebmarassing typoMarcelina Kościelnicka2021-11-101-1/+1
| | | |
| * | | iopadmap: Add native support for negative-polarity output enable.Marcelina Kościelnicka2021-11-091-7/+22
| | | |
| * | | gowin: widelut support (#3042)Pepijn de Vos2021-11-061-2/+8
| |/ /
| * | Make it work on allMiodrag Milanovic2021-11-051-2/+4
| | |
| * | Correct way of setting maybe_unsused on labelsMiodrag Milanovic2021-11-051-4/+2
| | |
| * | flatten: Keep sigmap around between flatten_cell invocations.Marcelina Kościelnicka2021-11-021-3/+4
| | | | | | | | | | | | Fixes #3064.
| * | proc_dff: Emit $aldff.Marcelina Kościelnicka2021-10-271-32/+7
| | |
| * | dfflegalize: Refactor, add aldff support.Marcelina Kościelnicka2021-10-271-973/+889
| | |
| * | verilog: use derived module info to elaborate cell connectionsZachary Snow2021-10-252-1/+7
| | | | | | | | | | | | | | | | | | | | | | | | - Attempt to lookup a derived module if it potentially contains a port connection with elaboration ambiguities - Mark the cell if module has not yet been derived - This can be extended to implement automatic hierarchical port connections in a future change
| * | Split out logic for reprocessing an AstModuleRupert Swarbrick2021-10-251-1/+1
| | | | | | | | | | | | | | | This will enable other features to use same core logic for replacing an existing AstModule with a newly elaborated version.
| * | Change implicit conversions from bool to Sig* to explicit.Marcelina Kościelnicka2021-10-211-4/+6
| | | | | | | | | | | | Also fixes some completely broken code in extract_reduce.
| * | extract_reduce: Refactor and fix input signal construction.Marcelina Kościelnicka2021-10-211-63/+34
| | | | | | | | | | | | Fixes #3047.
| * | dfflegalize: remove redundant check for initialized dlatchPaul Annesley2021-10-171-4/+0
| | | | | | | | | | | | | | | | | | This if condition is repeated verbatim, and I can't imagine a legitimate way the inputs could change in between. I imagine it's a copy/paste mistake.
| * | FfData: some refactoring.Marcelina Kościelnicka2021-10-077-87/+48
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - FfData now keeps track of the module and underlying cell, if any (so calling emit on FfData created from a cell will replace the existing cell) - FfData implementation is split off to its own .cc file for faster compilation - the "flip FF data sense by inserting inverters in front and after" functionality that zinit uses is moved onto FfData class and beefed up to have dffsr support, to support more use cases
| * | Hook up $aldff support in various passes.Marcelina Kościelnicka2021-10-023-4/+16
| | |
| * | zinit: Refactor to use FfData.Marcelina Kościelnicka2021-10-021-101/+38
| | |
| * | kernel/ff: Refactor FfData to enable FFs with async load.Marcelina Kościelnicka2021-10-025-130/+220
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - *_en is split into *_ce (clock enable) and *_aload (async load aka latch gate enable), so both can be present at once - has_d is removed - has_gclk is added (to have a clear marker for $ff) - d_is_const and val_d leftovers are removed - async2sync, clk2fflogic, opt_dff are updated to operate correctly on FFs with async load
| * | simplemap: refactor to use FfData.Marcelina Kościelnicka2021-10-022-287/+20
| | |
| * | abc9: make re-entrant (#2993)Eddie Hung2021-09-092-9/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | * Add testcase * Cleanup some state at end of abc9 * Re-assign abc9_box_id from scratch * Suppress delete unless prep_bypass did something
| * | abc9: holes module to instantiate cells with NEW_ID (#2992)Eddie Hung2021-09-091-1/+1
| | | | | | | | | | | | | | | * Add testcase * holes module to instantiate cells with NEW_ID
| * | abc9: replace cell type/parameters if derived type already processed (#2991)Eddie Hung2021-09-091-6/+22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Add close bracket * Add testcase * Replace cell type/param if in unmap_design * Improve abc9_box error message too * Update comment as per review
| * | opt_merge: Remove and reinsert init when connecting nets.Marcelina Kościelnicka2021-08-221-3/+4
| | | | | | | | | | | | | | | | | | | | | | | | Mutating the SigMap by adding a new connection will throw off FfInitVals index. Work around this by removing the relevant init values from index whenever we connect nets, then re-add the new init value. Should fix #2920.
| * | opt_clean: Make the init attribute follow the FF's Q.Marcelina Kościelnicka2021-08-221-0/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously, opt_clean would reconnect all ports (including FF Q ports) to a "canonical" SigBit chosen by complex rules, but would leave the init attribute on the old wire. This change applies the same canonicalization rules to the init attributes, ensuring that init moves to wherever the Q port moved. Part of another jab at #2920.
| * | proc_prune: Make assign removal and promotion per-bit, remember promoted bits.Marcelina Kościelnicka2021-08-141-40/+25
| | | | | | | | | | | | Fixes #2962.
| * | Add opt_mem_widen pass.Marcelina Kościelnicka2021-08-143-0/+110
| | | | | | | | | | | | If all of us are wide, then none of us are!
| * | memory_share: Add -nosat and -nowiden options.Marcelina Kościelnicka2021-08-142-10/+41
| | | | | | | | | | | | This unlocks wide port recognition by default.
| * | memory_dff: Recognize soft transparency logic.Marcelina Kościelnicka2021-08-131-7/+451
| | |
| * | Add new opt_mem_priority pass.Marcelina Kościelnicka2021-08-133-2/+114
| | |
| * | Merge pull request #2932 from YosysHQ/mwk/logger-check-expectedMiodrag Milanović2021-08-131-0/+9
| |\ \ | | | | | | | | logger: Add -check-expected subcommand.
| | * | logger: Add -check-expected subcommand.Marcelina Kościelnicka2021-08-121-0/+9
| | | | | | | | | | | | | | | | | | | | This allows us to have multiple "expect this warning" calls in a single long script, covering only as many passes as necessary.
| * | | memory_share: Pass addresses through sigmap_xmux everywhere.Marcelina Kościelnicka2021-08-131-20/+25
| |/ / | | | | | | | | | This fixes wide port recognition in some cases.
| * | memory_dff: Recognize read ports with reset / initial value.Marcelina Kościelnicka2021-08-111-7/+0
| | |
| * | proc_memwr: Use the v2 memwr cell.Marcelina Kościelnicka2021-08-111-9/+19
| | |
| * | Add v2 memory cells.Marcelina Kościelnicka2021-08-117-10/+14
| | |
| * | kernel/mem: Introduce transparency masks.Marcelina Kościelnicka2021-08-114-70/+45
| | |
| * | Refactor common parts of SAT-using optimizations into a helper.Marcelina Kościelnicka2021-08-093-150/+40
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This also aligns the functionality: - in all cases, the onehot attribute is used to create appropriate constraints (previously, opt_dff didn't do it at all, and share created one-hot constraints based on $pmux presence alone, which is unsound) - in all cases, shift and mul/div/pow cells are now skipped when importing the SAT problem (previously only memory_share did this) — this avoids creating clauses for hard cells that are unlikely to help with proving the UNSATness needed for optimization
| * | opt_merge: Use FfInitVals.Marcelina Kościelnicka2021-08-081-27/+8
| | | | | | | | | | | | Partial #2920 fix.
| * | memory_share: Don't skip ports with EN wired to input for SAT sharing.Marcelina Kościelnicka2021-08-041-3/+1
| | | | | | | | | | | | Fixes #2912.
| * | memory_bram: Move init data swizzling before other swizzling.Marcelina Kościelnicka2021-08-031-18/+18
| | | | | | | | | | | | Fixes #2907.
| * | memory_bram: Some refactoringMarcelina Kościelnicka2021-08-011-196/+174
| | | | | | | | | | | | | | | | | | This will make more sense when the new transparency masks land. Fixes #2902.