search

chipsalliance / Cores-VeeR-EH2

Apache License 2.0
211 stars 57 forks source link

unused lsu_cmpen_dc2 signal #28

Closed vignajeth closed 2 years ago

vignajeth commented 2 years ago

Hi, I think _lsu_cmpendc2 signal has no importance, https://github.com/chipsalliance/Cores-SweRV-EH2/blob/a95fdb81ea6dc19239cfe2953ef00710a6f1cf2a/design/lsu/eh2_lsu.sv#L392-L393

Even though _lsu_cmpendc2 is used in the below lines, the signals _cmpen_hidc2 , _cmpen_lodc2 are again unused

https://github.com/chipsalliance/Cores-SweRV-EH2/blob/a95fdb81ea6dc19239cfe2953ef00710a6f1cf2a/design/lsu/eh2_lsu_stbuf.sv#L319-L324

vignajeth commented 2 years ago

synthesis tool will take care of this