chipsalliance / Cores-VeeR-EL2

VeeR EL2 Core
https://chipsalliance.github.io/Cores-VeeR-EL2/html/
Apache License 2.0
243 stars 73 forks source link

rvdffppe elab issue #73

Closed howardtr closed 1 year ago

howardtr commented 1 year ago

rvdffppe does not elaborate with it's default WIDTH parameter + no defines.

https://github.com/chipsalliance/Cores-VeeR-EL2/blob/a1d96f9c1493be8b3a68f35ba61f3686e85ba1b5/design/lib/beh_lib.sv#L294

https://github.com/chipsalliance/Cores-VeeR-EL2/blob/a1d96f9c1493be8b3a68f35ba61f3686e85ba1b5/design/lib/beh_lib.sv#L304-L305

https://github.com/chipsalliance/Cores-VeeR-EL2/blob/a1d96f9c1493be8b3a68f35ba61f3686e85ba1b5/design/lib/beh_lib.sv#L327-L331

Proposal: Set WIDTH >=39 since LEFT = 39 - 31 = 8

This will allow the module to get elaborated independently. Note that this module works fine when included in the larger system since WIDTH is overwritten.

tmichalak commented 1 year ago

Fixed by #76