This is an archive of the discontinued LLVM Phabricator instance.

[RISCV] Remove experimental-b extension that includes all Zb* extensions
ClosedPublic

Authored by craig.topper on Oct 7 2021, 12:09 PM.

Details

Summary

At this point it looks like a B extension will never exist. Instead Zba, Zbb, Zbc, and Zbs are individual extensions being ratified together as package. Unclear what will happen with the other Zb* extensions from the 0.93 spec.

This patch removes references to the B extension. I've updated and split tests accordingly.

This has been split from D110669 to make review a little easier.

Diff Detail

Event Timeline

craig.topper created this revision.Oct 7 2021, 12:09 PM
craig.topper requested review of this revision.Oct 7 2021, 12:09 PM
Herald added a project: Restricted Project. · View Herald TranscriptOct 7 2021, 12:09 PM
Herald added a subscriber: MaskRay. · View Herald Transcript

Technically they're individual extensions being ratified together as part of a Zb[abcs] "package"

craig.topper edited the summary of this revision. (Show Details)Oct 7 2021, 12:21 PM
kito-cheng accepted this revision.Oct 7 2021, 7:44 PM

LGTM.

Of cause we can still keep this as 0.93 but it's weird since it included zba, zbb, zbc and zbs, which is 1.0 not 0.93, and might cause confusion...so remove that is make more sense to me .

This revision is now accepted and ready to land.Oct 7 2021, 7:44 PM
This revision was landed with ongoing or failed builds.Oct 7 2021, 8:50 PM
This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptOct 7 2021, 8:50 PM
Herald added a subscriber: cfe-commits. · View Herald Transcript
llvm/test/MC/RISCV/rv64zbs-aliases-valid.s