This is an archive of the discontinued LLVM Phabricator instance.

[RISCV] Switch vwsll.v* instructions to use the VSHT_IV_V_X_I class instead of VALU_IV_V_X_I.
ClosedPublic

Authored by craig.topper on Jun 2 2023, 1:53 PM.

Details

Summary

This allows us to remove the uimm5 argument and changes the
scheduler class from ALU to Shift.

Ultimately we need a WShift scheduler class, but we need to scrub
all of the crypto instructions for scheduler classes so I'll leave
that for future work.

Diff Detail

Event Timeline

craig.topper created this revision.Jun 2 2023, 1:53 PM
Herald added a project: Restricted Project. · View Herald TranscriptJun 2 2023, 1:53 PM
craig.topper requested review of this revision.Jun 2 2023, 1:53 PM
Herald added a project: Restricted Project. · View Herald TranscriptJun 2 2023, 1:53 PM
ego accepted this revision.Jun 5 2023, 8:38 AM

Looks good to me.

This revision is now accepted and ready to land.Jun 5 2023, 8:38 AM
4vtomat accepted this revision.Jun 5 2023, 9:06 AM

LGTM~