Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Merge pull request #1875 from whitequark/read_ilang-int_overflow | whitequark | 2020-04-09 | 2 | -2/+14 |
|\ | | | | | Improve handling of integer literals in RTLIL frontend | ||||
| * | read_ilang: improve style. NFC. | whitequark | 2020-04-06 | 1 | -2/+1 |
| | | | | | | Co-Authored-By: Alberto Gonzalez <61295559+boqwxp@users.noreply.github.com> | ||||
| * | read_ilang: improve error message for overly long wires. | whitequark | 2020-04-06 | 1 | -0/+3 |
| | | | | | | | | Fixes #1838. | ||||
| * | read_ilang: detect overflow of integer literals. | whitequark | 2020-04-06 | 1 | -1/+11 |
| | | |||||
* | | aigerparse: only define __STDC_FORMAT_MACROS it not already before. | Henner Zeller | 2020-04-07 | 1 | -0/+2 |
|/ | |||||
* | Merge pull request #1853 from YosysHQ/eddie/fix_dynslice | Eddie Hung | 2020-04-02 | 1 | -1/+2 |
|\ | | | | | ast: cap dynamic range select to size of signal, suppresses warnings | ||||
| * | ast: cap dynamic range select to size of signal, suppresses warnings | Eddie Hung | 2020-04-01 | 1 | -1/+2 |
| | | |||||
* | | Merge pull request #1767 from YosysHQ/eddie/idstrings | Eddie Hung | 2020-04-02 | 9 | -449/+437 |
|\ \ | | | | | | | IdString: use more ID::*, make them easier to use, speed up IdString::in() | ||||
| * | | kernel: big fat patch to use more ID::*, otherwise ID(*) | Eddie Hung | 2020-04-02 | 8 | -313/+301 |
| | | | |||||
| * | | kernel: use more ID::* | Eddie Hung | 2020-04-02 | 9 | -142/+142 |
| | | | |||||
* | | | Merge pull request #1846 from dh73/ast_fe | Claire Wolf | 2020-04-02 | 1 | -0/+3 |
|\ \ \ | |/ / |/| | | Adding error message for when size (width) of number literal is zero | ||||
| * | | Replacing log_error for log_file_error due consistency | Diego H | 2020-03-31 | 1 | -2/+1 |
| | | | |||||
| * | | Adding error message for when size (width) of number literal is zero | Diego H | 2020-03-30 | 1 | -0/+4 |
| | | | |||||
* | | | Merge pull request #1845 from YosysHQ/eddie/kernel_speedup | Eddie Hung | 2020-04-02 | 3 | -38/+44 |
|\ \ \ | | | | | | | | | kernel: speedup by using more pass-by-const-ref | ||||
| * | | | kernel: more pass by const ref, more speedups | Eddie Hung | 2020-03-18 | 3 | -38/+44 |
| | | | | |||||
* | | | | Merge pull request #1844 from YosysHQ/dave/gen-source-loc | David Shah | 2020-04-01 | 1 | -0/+6 |
|\ \ \ \ | | | | | | | | | | | verilog: Add location info for generate constructs | ||||
| * | | | | verilog: Add location info for generate constructs | David Shah | 2020-04-01 | 1 | -0/+6 |
| | | | | | | | | | | | | | | | | | | | | Signed-off-by: David Shah <dave@ds0.me> | ||||
* | | | | | Merge pull request #1848 from YosysHQ/eddie/fix_dynslice | Claire Wolf | 2020-04-01 | 1 | -1/+1 |
|\ \ \ \ \ | |_|_|/ / |/| | | / | | |_|/ | |/| | | ast: simplify to fully populate dynamic slicing case transformation | ||||
| * | | | ast: simplify to fully populate dynamic slicing case transformation | Eddie Hung | 2020-03-31 | 1 | -1/+1 |
| | |/ | |/| | |||||
* | | | Merge pull request #1783 from boqwxp/astcc_cleanup | Eddie Hung | 2020-03-30 | 1 | -13/+20 |
|\ \ \ | |_|/ |/| | | Clean up pseudo-private member usage in `frontends/ast/ast.cc`. | ||||
| * | | Add explanatory comment about inefficient wire removal and remove ↵ | Alberto Gonzalez | 2020-03-30 | 1 | -4/+8 |
| | | | | | | | | | | | | | | | | | | superfluous call to `fixup_ports()`. Co-Authored-By: Eddie Hung <eddie@fpgeh.com> | ||||
| * | | Revert over-aggressive change to a more modest cleanup. | Alberto Gonzalez | 2020-03-27 | 1 | -2/+3 |
| | | | |||||
| * | | Clean up pseudo-private member usage in `frontends/ast/ast.cc`. | Alberto Gonzalez | 2020-03-19 | 1 | -11/+13 |
| | | | |||||
* | | | Merge pull request #1811 from PeterCrozier/typedef_scope | N. Engelhardt | 2020-03-30 | 4 | -41/+81 |
|\ \ \ | | | | | | | | | Support module/package/interface/block scope for typedef names. | ||||
| * | | | Inline productions to follow house style. | Peter Crozier | 2020-03-27 | 1 | -33/+29 |
| | | | | |||||
| * | | | Error duplicate declarations of a typedef name in the same scope. | Peter Crozier | 2020-03-24 | 2 | -3/+11 |
| | | | | |||||
| * | | | Support module/package/interface/block scope for typedef names. | Peter Crozier | 2020-03-23 | 4 | -20/+56 |
| | | | | |||||
* | | | | Merge pull request #1778 from rswarbrick/sv-defines | N. Engelhardt | 2020-03-30 | 4 | -149/+578 |
|\ \ \ \ | | | | | | | | | | | Add support for SystemVerilog-style `define to Verilog frontend | ||||
| * | | | | Add support for SystemVerilog-style `define to Verilog frontend | Rupert Swarbrick | 2020-03-27 | 4 | -149/+578 |
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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 #1607 from whitequark/simplify-simplify-meminit | Claire Wolf | 2020-03-27 | 1 | -63/+82 |
|\ \ \ \ | |/ / / |/| | | | ast: avoid intermediate wires/assigns when lowering to AST_MEMINIT | ||||
| * | | | ast: avoid intermediate wires/assigns when lowering to AST_MEMINIT. | whitequark | 2020-02-07 | 1 | -65/+84 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Before this commit, every initial assignment to a memory generated two wires and four assigns in a process. For unknown reasons (I did not investigate), large amounts of assigns cause quadratic slowdown later in the AST frontend, in processAst/removeSignalFromCaseTree. As a consequence, common and reasonable Verilog code, such as: reg [`WIDTH:0] mem [0:`DEPTH]; integer i; initial for (i = 0; i <= `DEPTH; i++) mem[i] = 0; took extremely long time to be processed; around 80 s for a 8-wide, 8192-deep memory. After this commit, initial assignments where address and/or data are constant (after `generate`) do not incur the cost of intermediate wires; expressions like `mem[i+1]=i^(i<<1)` are considered constant. This results in speedups of orders of magnitude for common memory sizes; it now takes merely 0.4 s to process a 8-wide, 8192-deep memory, and only 5.8 s to process a 8-wide, 131072-deep one. As a bonus, this change also results in nontrivial speedups later in the synthesis pipeline, since pass sequencing issues meant that all of these intermediate wires were subject to transformations such as width reduction, even though they existed solely to be constant folded away in `memory_collect`. | ||||
* | | | | Simplify was not being called for packages. Broke typedef enums. | Peter Crozier | 2020-03-22 | 1 | -5/+8 |
| | | | | |||||
* | | | | Build pkg_user_types before parsing in case of changes in the design. | Peter Crozier | 2020-03-22 | 1 | -6/+3 |
| | | | | |||||
* | | | | Clear pkg_user_types if no packages following a 'design -reset-vlog'. | Peter | 2020-03-22 | 2 | -0/+5 |
| | | | | |||||
* | | | | Parser changes to support typedef. | Peter | 2020-03-22 | 4 | -10/+88 |
| | | | | |||||
* | | | | Merge pull request #1788 from YosysHQ/eddie/fix_ndebug | Eddie Hung | 2020-03-19 | 2 | -2/+2 |
|\ \ \ \ | | | | | | | | | | | Fix NDEBUG warnings | ||||
| * | | | | Fix NDEBUG warnings | Eddie Hung | 2020-03-19 | 2 | -2/+2 |
| | | | | | |||||
* | | | | | Merge pull request #1787 from YosysHQ/mmicko/lexer_deps | Miodrag Milanović | 2020-03-19 | 1 | -1/+1 |
|\ \ \ \ \ | |/ / / / |/| | | | | Add dependency to verilog_lexer.cc | ||||
| * | | | | Add one mode dependency | Miodrag Milanovic | 2020-03-19 | 1 | -1/+1 |
| | |/ / | |/| | | |||||
* | | | | Merge pull request #1775 from huaixv/asserts_locations | N. Engelhardt | 2020-03-19 | 2 | -7/+31 |
|\ \ \ \ | |/ / / |/| | | | Add precise locations for asserts | ||||
| * | | | Add precise locations for asserts | huaixv | 2020-03-19 | 2 | -7/+31 |
| | | | | |||||
* | | | | Add AST node source location information in a couple more parser rules. | Alberto Gonzalez | 2020-03-17 | 1 | -0/+2 |
|/ / / | |||||
* | | | Merge pull request #1759 from zeldin/constant_with_comment_redux | Miodrag Milanović | 2020-03-14 | 2 | -19/+43 |
|\ \ \ | | | | | | | | | refixed parsing of constant with comment between size and value | ||||
| * | | | refixed parsing of constant with comment between size and value | Marcus Comstedt | 2020-03-11 | 2 | -19/+43 |
| | | | | | | | | | | | | | | | | | | | | | | | | The three parts of a based constant (size, base, digits) are now three separate tokens, allowing the linear whitespace (including comments) between them to be treated as normal inter-token whitespace. | ||||
* | | | | Merge pull request #1754 from boqwxp/precise_locations | Miodrag Milanović | 2020-03-14 | 1 | -2/+53 |
|\ \ \ \ | |_|_|/ |/| | | | Set AST node source location in more parser rules. | ||||
| * | | | verilog: also set location for simple_behavioral_stmt | Eddie Hung | 2020-03-10 | 1 | -0/+4 |
| | | | | |||||
| * | | | Set AST source locations in more parser rules. | Alberto Gonzalez | 2020-03-10 | 1 | -2/+49 |
| |/ / | |||||
* / / | Fix compilation for emcc | jiegec | 2020-03-11 | 1 | -1/+2 |
|/ / | |||||
* | | Fix partsel expr bit width handling and add test case | Claire Wolf | 2020-03-08 | 1 | -4/+6 |
| | | | | | | | | Signed-off-by: Claire Wolf <claire@symbioticeda.com> | ||||
* | | Fix bison warning for "pure-parser" option | Claire Wolf | 2020-03-03 | 1 | -1/+1 |
| | | | | | | | | Signed-off-by: Claire Wolf <claire@symbioticeda.com> |