[dv] Add RV32ZC handling to ISA strings#2398
Conversation
Use the RV32ZC configuration parameter when constructing the ISA string passed to the cosimulator. This lets DV runs distinguish between Zca, ZcaZcb, ZcaZcmp and ZcaZcbZcmp configurations.
|
Hi, @nasahlpa! |
|
Thanks @AnastasiyaChernikova for looking into this. Your changes look good at first glance to me. However, we won't be able to merge them until we update our compiler version. Some changes are similar to commits in the PR adding DV for the Zc* extensions (#2324), but as noted there, this cannot be merged until we update the compiler. For more context please also check out this comment on why the I am afraid we cannot merge this currently, but I'll leave the PR open and revisit it once we update the compiler. |
|
@SamuelRiedel, thanks for the explanation and for keeping the PR open. I understand that this is currently blocked by the toolchain/compiler version used in CI rather than by the patch itself, and I would like to help unblock it if possible. |
|
Hi @SamuelRiedel, sorry for the ping. Just gently following up on this PR. Is there any update on the toolchain/compiler blocker, or anything I could help with from my side? This is still relevant for my current work, so I would appreciate any guidance on the next steps. Thanks! |
Use the RV32ZC configuration parameter when constructing the ISA string passed to the cosimulator. This lets DV runs distinguish between Zca, ZcaZcb, ZcaZcmp and ZcaZcbZcmp configurations.