[dv] Add RV32ZC handling to ISA strings#2398
[dv] Add RV32ZC handling to ISA strings#2398AnastasiyaChernikova wants to merge 1 commit intolowRISC:masterfrom
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. |
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.