YosysHQ / yosys

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

tests: remove -seq 1 from sat with -tempinduct where possible #4629

Open georgerennie opened 1 month ago

georgerennie commented 1 month ago

When used with -tempinduct mode, -seq causes assertions to be ignored in the first N steps. While this has uses for reset modelling, for these test cases it is unnecessary and could lead to failures slipping through uncaught.

I suspect most of these have slipped through as they have just been copy-pasted from existing tests - Could be good to clarify the wording in the sat help to make it clear how this behaviour works