Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

"set: No match." error in yosys.sh #21

Open
xphoniex opened this issue Mar 1, 2021 · 2 comments
Open

"set: No match." error in yosys.sh #21

xphoniex opened this issue Mar 1, 2021 · 2 comments

Comments

@xphoniex
Copy link

xphoniex commented Mar 1, 2021

I'm trying to synthesize this processor, specifically this build.

My flow is failing right at Synthesis with message:

$ ./qflow_exec.sh
set: No match.

After diving into yosys.sh, the culprit seems to be yosys.sh#L379. If I add this line before that:

   ${bindir}/yosys -s ${modulename}.ys |& sed -e "/\\/s#\\#/#g" -e "/\[/s/\[//g" -e "/\]/s/\]//g" | grep ERROR
   # Note:  Remove backslashes and brackets to avoid problems with tcsh
   set yerrors = `eval ${bindir}/yosys -s ${modulename}.ys |& sed -e "/\\/s#\\#/#g" \
                -e "/\[/s/\[//g" -e "/\]/s/\]//g" | grep ERROR`

...

and run my script again, my output becomes:

$ ./qflow_exec.sh
ERROR: Module `/FIFO20' referenced in module `/mkCPU' in cell `/stage3_f_reset_rsps' is not part of the design.
ERROR: Module `/mkRISCV_MBox' referenced in module `/mkCPU' in cell `/stage2_mbox' is not part of the design.
ERROR: Module `/mkFBox_Top' referenced in module `/mkCPU' in cell `/stage2_fbox' is not part of the design.
ERROR: Module `/mkSoC_Map' referenced in module `/mkCPU' in cell `/soc_map' is not part of the design.
ERROR: Module `/mkNear_Mem' referenced in module `/mkCPU' in cell `/near_mem' is not part of the design.
ERROR: Module `/mkGPR_RegFile' referenced in module `/mkCPU' in cell `/gpr_regfile' is not part of the design.
ERROR: Module `/mkFPR_RegFile' referenced in module `/mkCPU' in cell `/fpr_regfile' is not part of the design.
ERROR: Module `/FIFO2' referenced in module `/mkCPU' in cell `/f_reset_rsps' is not part of the design.
ERROR: Module `/mkCSR_RegFile' referenced in module `/mkCPU' in cell `/csr_regfile' is not part of the design.
ERROR: Module `/mkCSR_MIP' referenced in module `/mkCSR_RegFile' in cell `/csr_mip' is not part of the design.
ERROR: Module `/mkCSR_MIE' referenced in module `/mkCSR_RegFile' in cell `/csr_mie' is not part of the design.
ERROR: Module `/RegFile' referenced in module `/mkFPR_RegFile' in cell `/regfile' is not part of the design.
ERROR: Module `/mkMMU_Cache' referenced in module `/mkNear_Mem' in cell `/icache' is not part of the design.
use of @* instead of @(...) for better match of ERROR: Module `/mkFBox_Core' referenced in module `/mkFBox_Top' in cell `/fbox_core' is not part of the design.
set: No match.

This seems a problem on part of tcsh and can(?) be solved if we stop piping stderr so |& becomes |, yet I'm still not able to synthesize because _mapped.v file does not exist. Full output after change:

$ ./qflow_exec.sh
ERROR: Module `\FIFO20' referenced in module `\mkCPU' in cell `\stage3_f_reset_rsps' is not part of the design.
ERROR: Module `\FIFO20' referenced in module `\mkCPU' in cell `\stage3_f_reset_rsps' is not part of the design.
Running yosys for verilog parsing and synthesis
yosys  -s mkCPU.ys

 /----------------------------------------------------------------------------\
 |                                                                            |
 |  yosys -- Yosys Open SYnthesis Suite                                       |
 |                                                                            |
 |  Copyright (C) 2012 - 2016  Clifford Wolf <[email protected]>           |
 |                                                                            |
 |  Permission to use, copy, modify, and/or distribute this software for any  |
 |  purpose with or without fee is hereby granted, provided that the above    |
 |  copyright notice and this permission notice appear in all copies.         |
 |                                                                            |
 |  THE SOFTWARE IS PROVIDED "AS IS" AND THE AUTHOR DISCLAIMS ALL WARRANTIES  |
 |  WITH REGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED WARRANTIES OF          |
 |  MERCHANTABILITY AND FITNESS. IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR   |
 |  ANY SPECIAL, DIRECT, INDIRECT, OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES    |
 |  WHATSOEVER RESULTING FROM LOSS OF USE, DATA OR PROFITS, WHETHER IN AN     |
 |  ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF   |
 |  OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.            |
 |                                                                            |
 \----------------------------------------------------------------------------/

 Yosys 0.7 (git sha1 61f6811, gcc 6.2.0-11ubuntu1 -O2 -fdebug-prefix-map=/build/yosys-OIL3SR/yosys-0.7=. -fstack-protector-strong -fPIC -Os)


-- Executing script file `mkCPU.ys' --

1. Executing Liberty frontend.
Imported 32 cell types from liberty file.

2. Executing Verilog-2005 frontend.
Parsing Verilog input from `/.../qflow-piccolo/source/mkCPU.v' to AST representation.
Warning: Found one of those horrible `(synopsys|synthesis) parallel_case' comments.
Yosys does support them but it is recommended to use Verilog `parallel_case' attributes instead!
Warning: Found one of those horrible `(synopsys|synthesis) translate_off' comments.
Yosys does support them but it is recommended to use `ifdef constructs instead!
Generating RTLIL representation for module `\mkCPU'.
Note: Assuming pure combinatorial block at /.../qflow-piccolo/source/mkCPU.v:2432 in
compliance with IEC 62142(E):2005 / IEEE Std. 1364.1(E):2002. Recommending
use of @* instead of @(...) for better match of synthesis and simulation.
Note: Assuming pure combinatorial block at /.../qflow-piccolo/source/mkCPU.v:2513 in
compliance with IEC 62142(E):2005 / IEEE Std. 1364.1(E):2002. Recommending
use of @* instead of @(...) for better match of synthesis and simulation.
Note: Assuming pure combinatorial block at /.../qflow-piccolo/source/mkCPU.v:2591 in
compliance with IEC 62142(E):2005 / IEEE Std. 1364.1(E):2002. Recommending
use of @* instead of @(...) for better match of synthesis and simulation.
Note: Assuming pure combinatorial block at /.../qflow-piccolo/source/mkCPU.v:2628 in
compliance with IEC 62142(E):2005 / IEEE Std. 1364.1(E):2002. Recommending
use of @* instead of @(...) for better match of synthesis and simulation.
Note: Assuming pure combinatorial block at /.../qflow-piccolo/source/mkCPU.v:2660 in
compliance with IEC 62142(E):2005 / IEEE Std. 1364.1(E):2002. Recommending
use of @* instead of @(...) for better match of synthesis and simulation.
Note: Assuming pure combinatorial block at /.../qflow-piccolo/source/mkCPU.v:2708 in
compliance with IEC 62142(E):2005 / IEEE Std. 1364.1(E):2002. Recommending
use of @* instead of @(...) for better match of synthesis and simulation.
Note: Assuming pure combinatorial block at /.../qflow-piccolo/source/mkCPU.v:2762 in
compliance with IEC 62142(E):2005 / IEEE Std. 1364.1(E):2002. Recommending
use of @* instead of @(...) for better match of synthesis and simulation.
Note: Assuming pure combinatorial block at /.../qflow-piccolo/source/mkCPU.v:2809 in
compliance with IEC 62142(E):2005 / IEEE Std. 1364.1(E):2002. Recommending
use of @* instead of @(...) for better match of synthesis and simulation.
Note: Assuming pure combinatorial block at /.../qflow-piccolo/source/mkCPU.v:2876 in
compliance with IEC 62142(E):2005 / IEEE Std. 1364.1(E):2002. Recommending
use of @* instead of @(...) for better match of synthesis and simulation.
Note: Assuming pure combinatorial block at /.../qflow-piccolo/source/mkCPU.v:2926 in
compliance with IEC 62142(E):2005 / IEEE Std. 1364.1(E):2002. Recommending
use of @* instead of @(...) for better match of synthesis and simulation.
Note: Assuming pure combinatorial block at /.../qflow-piccolo/source/mkCPU.v:2934 in
compliance with IEC 62142(E):2005 / IEEE Std. 1364.1(E):2002. Recommending
use of @* instead of @(...) for better match of synthesis and simulation.
Note: Assuming pure combinatorial block at /.../qflow-piccolo/source/mkCPU.v:2964 in
compliance with IEC 62142(E):2005 / IEEE Std. 1364.1(E):2002. Recommending
use of @* instead of @(...) for better match of synthesis and simulation.
Note: Assuming pure combinatorial block at /.../qflow-piccolo/source/mkCPU.v:3108 in
compliance with IEC 62142(E):2005 / IEEE Std. 1364.1(E):2002. Recommending
use of @* instead of @(...) for better match of synthesis and simulation.
Note: Assuming pure combinatorial block at /.../qflow-piccolo/source/mkCPU.v:3134 in
compliance with IEC 62142(E):2005 / IEEE Std. 1364.1(E):2002. Recommending
use of @* instead of @(...) for better match of synthesis and simulation.
Note: Assuming pure combinatorial block at /.../qflow-piccolo/source/mkCPU.v:3163 in
compliance with IEC 62142(E):2005 / IEEE Std. 1364.1(E):2002. Recommending
use of @* instead of @(...) for better match of synthesis and simulation.
Note: Assuming pure combinatorial block at /.../qflow-piccolo/source/mkCPU.v:3196 in
compliance with IEC 62142(E):2005 / IEEE Std. 1364.1(E):2002. Recommending
use of @* instead of @(...) for better match of synthesis and simulation.
Note: Assuming pure combinatorial block at /.../qflow-piccolo/source/mkCPU.v:4695 in
compliance with IEC 62142(E):2005 / IEEE Std. 1364.1(E):2002. Recommending
use of @* instead of @(...) for better match of synthesis and simulation.
Note: Assuming pure combinatorial block at /.../qflow-piccolo/source/mkCPU.v:4702 in
compliance with IEC 62142(E):2005 / IEEE Std. 1364.1(E):2002. Recommending
use of @* instead of @(...) for better match of synthesis and simulation.
Note: Assuming pure combinatorial block at /.../qflow-piccolo/source/mkCPU.v:4710 in
compliance with IEC 62142(E):2005 / IEEE Std. 1364.1(E):2002. Recommending
use of @* instead of @(...) for better match of synthesis and simulation.
Note: Assuming pure combinatorial block at /.../qflow-piccolo/source/mkCPU.v:4717 in
compliance with IEC 62142(E):2005 / IEEE Std. 1364.1(E):2002. Recommending
use of @* instead of @(...) for better match of synthesis and simulation.
Note: Assuming pure combinatorial block at /.../qflow-piccolo/source/mkCPU.v:4725 in
compliance with IEC 62142(E):2005 / IEEE Std. 1364.1(E):2002. Recommending
use of @* instead of @(...) for better match of synthesis and simulation.
Note: Assuming pure combinatorial block at /.../qflow-piccolo/source/mkCPU.v:4734 in
compliance with IEC 62142(E):2005 / IEEE Std. 1364.1(E):2002. Recommending
use of @* instead of @(...) for better match of synthesis and simulation.
Note: Assuming pure combinatorial block at /.../qflow-piccolo/source/mkCPU.v:4741 in
compliance with IEC 62142(E):2005 / IEEE Std. 1364.1(E):2002. Recommending
use of @* instead of @(...) for better match of synthesis and simulation.
Note: Assuming pure combinatorial block at /.../qflow-piccolo/source/mkCPU.v:4748 in
compliance with IEC 62142(E):2005 / IEEE Std. 1364.1(E):2002. Recommending
use of @* instead of @(...) for better match of synthesis and simulation.
Note: Assuming pure combinatorial block at /.../qflow-piccolo/source/mkCPU.v:4760 in
compliance with IEC 62142(E):2005 / IEEE Std. 1364.1(E):2002. Recommending
use of @* instead of @(...) for better match of synthesis and simulation.
Note: Assuming pure combinatorial block at /.../qflow-piccolo/source/mkCPU.v:4772 in
compliance with IEC 62142(E):2005 / IEEE Std. 1364.1(E):2002. Recommending
use of @* instead of @(...) for better match of synthesis and simulation.
Note: Assuming pure combinatorial block at /.../qflow-piccolo/source/mkCPU.v:4783 in
compliance with IEC 62142(E):2005 / IEEE Std. 1364.1(E):2002. Recommending
use of @* instead of @(...) for better match of synthesis and simulation.
Note: Assuming pure combinatorial block at /.../qflow-piccolo/source/mkCPU.v:4800 in
compliance with IEC 62142(E):2005 / IEEE Std. 1364.1(E):2002. Recommending
use of @* instead of @(...) for better match of synthesis and simulation.
Note: Assuming pure combinatorial block at /.../qflow-piccolo/source/mkCPU.v:4815 in
compliance with IEC 62142(E):2005 / IEEE Std. 1364.1(E):2002. Recommending
use of @* instead of @(...) for better match of synthesis and simulation.
Note: Assuming pure combinatorial block at /.../qflow-piccolo/source/mkCPU.v:4830 in
compliance with IEC 62142(E):2005 / IEEE Std. 1364.1(E):2002. Recommending
use of @* instead of @(...) for better match of synthesis and simulation.
Note: Assuming pure combinatorial block at /.../qflow-piccolo/source/mkCPU.v:4843 in
compliance with IEC 62142(E):2005 / IEEE Std. 1364.1(E):2002. Recommending
use of @* instead of @(...) for better match of synthesis and simulation.
Note: Assuming pure combinatorial block at /.../qflow-piccolo/source/mkCPU.v:4856 in
compliance with IEC 62142(E):2005 / IEEE Std. 1364.1(E):2002. Recommending
use of @* instead of @(...) for better match of synthesis and simulation.
Note: Assuming pure combinatorial block at /.../qflow-piccolo/source/mkCPU.v:4876 in
compliance with IEC 62142(E):2005 / IEEE Std. 1364.1(E):2002. Recommending
use of @* instead of @(...) for better match of synthesis and simulation.
Note: Assuming pure combinatorial block at /.../qflow-piccolo/source/mkCPU.v:4892 in
compliance with IEC 62142(E):2005 / IEEE Std. 1364.1(E):2002. Recommending
use of @* instead of @(...) for better match of synthesis and simulation.
Note: Assuming pure combinatorial block at /.../qflow-piccolo/source/mkCPU.v:4910 in
compliance with IEC 62142(E):2005 / IEEE Std. 1364.1(E):2002. Recommending
use of @* instead of @(...) for better match of synthesis and simulation.
Note: Assuming pure combinatorial block at /.../qflow-piccolo/source/mkCPU.v:4923 in
compliance with IEC 62142(E):2005 / IEEE Std. 1364.1(E):2002. Recommending
use of @* instead of @(...) for better match of synthesis and simulation.
Note: Assuming pure combinatorial block at /.../qflow-piccolo/source/mkCPU.v:4949 in
compliance with IEC 62142(E):2005 / IEEE Std. 1364.1(E):2002. Recommending
use of @* instead of @(...) for better match of synthesis and simulation.
Note: Assuming pure combinatorial block at /.../qflow-piccolo/source/mkCPU.v:4975 in
compliance with IEC 62142(E):2005 / IEEE Std. 1364.1(E):2002. Recommending
use of @* instead of @(...) for better match of synthesis and simulation.
Note: Assuming pure combinatorial block at /.../qflow-piccolo/source/mkCPU.v:5075 in
compliance with IEC 62142(E):2005 / IEEE Std. 1364.1(E):2002. Recommending
use of @* instead of @(...) for better match of synthesis and simulation.
Note: Assuming pure combinatorial block at /.../qflow-piccolo/source/mkCPU.v:5121 in
compliance with IEC 62142(E):2005 / IEEE Std. 1364.1(E):2002. Recommending
use of @* instead of @(...) for better match of synthesis and simulation.
Note: Assuming pure combinatorial block at /.../qflow-piccolo/source/mkCPU.v:5146 in
compliance with IEC 62142(E):2005 / IEEE Std. 1364.1(E):2002. Recommending
use of @* instead of @(...) for better match of synthesis and simulation.
Note: Assuming pure combinatorial block at /.../qflow-piccolo/source/mkCPU.v:5174 in
compliance with IEC 62142(E):2005 / IEEE Std. 1364.1(E):2002. Recommending
use of @* instead of @(...) for better match of synthesis and simulation.
Note: Assuming pure combinatorial block at /.../qflow-piccolo/source/mkCPU.v:5253 in
compliance with IEC 62142(E):2005 / IEEE Std. 1364.1(E):2002. Recommending
use of @* instead of @(...) for better match of synthesis and simulation.
Note: Assuming pure combinatorial block at /.../qflow-piccolo/source/mkCPU.v:5299 in
compliance with IEC 62142(E):2005 / IEEE Std. 1364.1(E):2002. Recommending
use of @* instead of @(...) for better match of synthesis and simulation.
Note: Assuming pure combinatorial block at /.../qflow-piccolo/source/mkCPU.v:5324 in
compliance with IEC 62142(E):2005 / IEEE Std. 1364.1(E):2002. Recommending
use of @* instead of @(...) for better match of synthesis and simulation.
Note: Assuming pure combinatorial block at /.../qflow-piccolo/source/mkCPU.v:5351 in
compliance with IEC 62142(E):2005 / IEEE Std. 1364.1(E):2002. Recommending
use of @* instead of @(...) for better match of synthesis and simulation.
Note: Assuming pure combinatorial block at /.../qflow-piccolo/source/mkCPU.v:5359 in
compliance with IEC 62142(E):2005 / IEEE Std. 1364.1(E):2002. Recommending
use of @* instead of @(...) for better match of synthesis and simulation.
Note: Assuming pure combinatorial block at /.../qflow-piccolo/source/mkCPU.v:5370 in
compliance with IEC 62142(E):2005 / IEEE Std. 1364.1(E):2002. Recommending
use of @* instead of @(...) for better match of synthesis and simulation.
Note: Assuming pure combinatorial block at /.../qflow-piccolo/source/mkCPU.v:5390 in
compliance with IEC 62142(E):2005 / IEEE Std. 1364.1(E):2002. Recommending
use of @* instead of @(...) for better match of synthesis and simulation.
Note: Assuming pure combinatorial block at /.../qflow-piccolo/source/mkCPU.v:5404 in
compliance with IEC 62142(E):2005 / IEEE Std. 1364.1(E):2002. Recommending
use of @* instead of @(...) for better match of synthesis and simulation.
Note: Assuming pure combinatorial block at /.../qflow-piccolo/source/mkCPU.v:5412 in
compliance with IEC 62142(E):2005 / IEEE Std. 1364.1(E):2002. Recommending
use of @* instead of @(...) for better match of synthesis and simulation.
Note: Assuming pure combinatorial block at /.../qflow-piccolo/source/mkCPU.v:5426 in
compliance with IEC 62142(E):2005 / IEEE Std. 1364.1(E):2002. Recommending
use of @* instead of @(...) for better match of synthesis and simulation.
Note: Assuming pure combinatorial block at /.../qflow-piccolo/source/mkCPU.v:5495 in
compliance with IEC 62142(E):2005 / IEEE Std. 1364.1(E):2002. Recommending
use of @* instead of @(...) for better match of synthesis and simulation.
Note: Assuming pure combinatorial block at /.../qflow-piccolo/source/mkCPU.v:5521 in
compliance with IEC 62142(E):2005 / IEEE Std. 1364.1(E):2002. Recommending
use of @* instead of @(...) for better match of synthesis and simulation.
Note: Assuming pure combinatorial block at /.../qflow-piccolo/source/mkCPU.v:5535 in
compliance with IEC 62142(E):2005 / IEEE Std. 1364.1(E):2002. Recommending
use of @* instead of @(...) for better match of synthesis and simulation.
Note: Assuming pure combinatorial block at /.../qflow-piccolo/source/mkCPU.v:5549 in
compliance with IEC 62142(E):2005 / IEEE Std. 1364.1(E):2002. Recommending
use of @* instead of @(...) for better match of synthesis and simulation.
Note: Assuming pure combinatorial block at /.../qflow-piccolo/source/mkCPU.v:5562 in
compliance with IEC 62142(E):2005 / IEEE Std. 1364.1(E):2002. Recommending
use of @* instead of @(...) for better match of synthesis and simulation.
Note: Assuming pure combinatorial block at /.../qflow-piccolo/source/mkCPU.v:5585 in
compliance with IEC 62142(E):2005 / IEEE Std. 1364.1(E):2002. Recommending
use of @* instead of @(...) for better match of synthesis and simulation.
Note: Assuming pure combinatorial block at /.../qflow-piccolo/source/mkCPU.v:5601 in
compliance with IEC 62142(E):2005 / IEEE Std. 1364.1(E):2002. Recommending
use of @* instead of @(...) for better match of synthesis and simulation.
Note: Assuming pure combinatorial block at /.../qflow-piccolo/source/mkCPU.v:5616 in
compliance with IEC 62142(E):2005 / IEEE Std. 1364.1(E):2002. Recommending
use of @* instead of @(...) for better match of synthesis and simulation.
Note: Assuming pure combinatorial block at /.../qflow-piccolo/source/mkCPU.v:5651 in
compliance with IEC 62142(E):2005 / IEEE Std. 1364.1(E):2002. Recommending
use of @* instead of @(...) for better match of synthesis and simulation.
Note: Assuming pure combinatorial block at /.../qflow-piccolo/source/mkCPU.v:5663 in
compliance with IEC 62142(E):2005 / IEEE Std. 1364.1(E):2002. Recommending
use of @* instead of @(...) for better match of synthesis and simulation.
Successfully finished Verilog frontend.

3. Executing SYNTH pass.

3.1. Executing HIERARCHY pass (managing design hierarchy).

3.1.1. Analyzing design hierarchy..
ERROR: Module `\FIFO20' referenced in module `\mkCPU' in cell `\stage3_f_reset_rsps' is not part of the design.
outputprep failure:  No file mkCPU_mapped.v.
Premature exit.

Any help would be appreciated.
@xphoniex
Copy link
Author

xphoniex commented Mar 3, 2021

So inside tcsh running:

> /usr/local/share/qflow/bin/yosys -s mkCPU.ys 

.
.
.
.
.

Note: Assuming pure combinatorial block at mkMMU_Cache.v:4692 in
compliance with IEC 62142(E):2005 / IEEE Std. 1364.1(E):2002. Recommending
use of @* instead of @(...) for better match of synthesis and simulation.
ERROR: Module `\mkFBox_Core' referenced in module `\mkFBox_Top' in cell `\fbox_core' is not part of the design.

but somehow grep modifies it a bit:

> /usr/local/share/qflow/bin/yosys -s mkCPU.ys | & grep ERROR
use of @* instead of @(...) for better match of ERROR: Module `\mkFBox_Core' referenced in module `\mkFBox_Top' in cell `\fbox_core' is not part of the design.

(where did the synthesis and simulation. part go?)

Anyway I kinda cheated by removing anything extra before 'ERROR' and it works now:

set yerrors = `eval ${bindir}/yosys -s ${modulename}.ys |& sed -e "/\\/s#\\#/#g" \
                -e "/\[/s/\[//g" -e "/\]/s/\]//g" | grep ERROR | sed 's/^.*\(ERROR.*\).*$/\1/'`

Can you please take a look @RTimothyEdwards to make sure this doesn't break anything else? I can make a patch.

@RTimothyEdwards
Copy link
Owner

Honestly, I think the answer to annoying issues with tcsh and other annoying issue with bash is to stop using shell scripts altogether and rewrite everything in python.

The problem here is that "|&" does not, as I would like, direct only stderr to the pipe; you can only either redirect stdout or both stdout and stderr together. Because they are output together but buffered separately, there is no guarantee that stderr and stdout won't be interleaved. Really, the best solution is to do it in python, where one can call subprocess() and then handle the stdout and stderr streams separately.

Your solution seems reasonable, so I'll adopt it for now.

RTimothyEdwards added a commit that referenced this issue Mar 5, 2021
@RTimothyEdwards
Modified yosys.sy according to the correction by Dave (xphoniex)
67b1dfe 
in github issue #21.  The problem is that the "|&" in the tcsh
script combines stdout and stderr, and because stdout is not
flushed by yosys, both come out randomly interleaved.  So in parsing
a line for "ERROR" one cannot assume that the line begins with
"ERROR".  The correction strips off anything in front of "ERROR"
so that the subsequent parsing captures the text of the error
message as intended.  As I noted in a comment to the issue, the
shell scripts should really all be rewritten in python.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@RTimothyEdwards @xphoniex