YosysHQ / yosys

Yosys Open SYnthesis Suite
https://yosyshq.net/yosys/
ISC License
3.49k stars 893 forks source link

Reduce default severity of Verific messages that produce warnings on commonly used coding styles #4324

Open nakengelhardt opened 7 months ago

nakengelhardt commented 7 months ago

In common with most EDA tools, Verific produces warnings on a lot of benign, commonly used (System-)Verilog coding styles. This is not in line with the Yosys philosophy, where warnings usually indicate a significant problem in the design that is likely to result in missynthesis.

The following messages should have their default severity decreased from WARNING to INFO:

As a motivating data point, using verific -set-info on the first three reduced the warning load from 176 mostly useless warnings to 10 very relevant warnings on the BlackParrot core.

nakengelhardt commented 1 week ago

Related but the inverse of the title, increase the severity of the following messages from WARNING to ERROR:

"latch inferred for net '%s'", "VERI-2580"
"latch inferred for net '%s'", "VHDL-1840"