This is an archive of the discontinued LLVM Phabricator instance.

[RISCV][NFC] Replace explicit type i64 with riscv customized SDTypeProfile.
ClosedPublic

Authored by Jim on Apr 8 2021, 4:20 AM.

Details

Summary

New SDTypeProfile can be reused for other word operation patterns without explicit i64 type in the future.

Diff Detail

Event Timeline

Jim created this revision.Apr 8 2021, 4:20 AM
Jim requested review of this revision.Apr 8 2021, 4:20 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 8 2021, 4:20 AM
craig.topper added inline comments.Apr 8 2021, 9:24 AM
llvm/lib/Target/RISCV/RISCVInstrInfo.td
43

Since the types of both operands and the result have to be i64 isn't this equivalent to SDT_RISCVIntBinOpW?

Jim updated this revision to Diff 336302.Apr 8 2021, 8:00 PM

Remove duplicate SDT_RISCVIntShiftOpW.

Jim marked an inline comment as done.Apr 8 2021, 8:04 PM
This revision is now accepted and ready to land.Apr 8 2021, 10:42 PM