I've removed the Zbs W instructions that are not part of the frozen spec.
References to B as an extension name have been removed. Tests are updated or split accordingly.
Paths
| Differential D110669
[RISCV] Update Zba, Zbb, Zbc, and Zbs version from 0.93 to 1.0. ClosedPublic Authored by craig.topper on Sep 28 2021, 5:29 PM.
Details Summary I've removed the Zbs W instructions that are not part of the frozen spec. References to B as an extension name have been removed. Tests are updated or split accordingly.
Diff Detail
Unit TestsFailed Event TimelineHerald added subscribers: achieveartificialintelligence, StephenFan, vkmr and 22 others. · View Herald TranscriptSep 28 2021, 5:29 PM Comment Actions We might need to update the version output for ELF attribute too, but I would prefer waiting https://reviews.llvm.org/D105168 instead of bump version in many place every time... Comment Actions
Done. Waiting for D105168 wouldn't prevent needing to update the tests so I went ahead and did it. Comment Actions
I think so. Comment Actions Drop experimental-b. Update tests accordingly. Comment Actions Would it be better to break this patch down into smaller parts. Maybe removing +experimental-b and the associated test changes first?
This revision is now accepted and ready to land.Oct 14 2021, 5:26 AM Closed by commit rGf7ba572483dd: [RISCV] Update Zba, Zbb, Zbc, and Zbs version from 0.93 to 1.0. (authored by craig.topper). · Explain WhyOct 14 2021, 9:25 AM This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 375746 clang/lib/Basic/Targets/RISCV.cpp
clang/lib/Driver/ToolChains/Arch/RISCV.cpp
clang/test/Driver/riscv-arch.c
clang/test/Preprocessor/riscv-target-features.c
llvm/lib/Target/RISCV/RISCVInstrInfoB.td
|
clang-format: please reformat the code