This is an archive of the discontinued LLVM Phabricator instance.

[RISCV] Remove SRO* and SLO* instructions from bitmanip.
ClosedPublic

Authored by craig.topper on Feb 5 2021, 10:02 AM.

Details

Summary

As of the current draft these are no longer being considered
for the bitmanip spec. It wasn't clear what sub extension they
belonged in in the 0.93 spec.

So remove them. They can always be added back if something changes.

Diff Detail

Event Timeline

craig.topper created this revision.Feb 5 2021, 10:02 AM
craig.topper requested review of this revision.Feb 5 2021, 10:02 AM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 5 2021, 10:02 AM
Herald added a subscriber: MaskRay. · View Herald Transcript

Looks like LLVM.MC/RISCV::rv(32|64)zbp-(in)?valid.s need updating too.

Update the MC tests.

frasercrmck accepted this revision.Feb 9 2021, 2:14 AM

LGTM.

Would this be appropriate to be cherry-picked into the release branch?

This revision is now accepted and ready to land.Feb 9 2021, 2:14 AM
This revision was landed with ongoing or failed builds.Feb 9 2021, 9:35 AM
This revision was automatically updated to reflect the committed changes.