This is an archive of the discontinued LLVM Phabricator instance.

[RISC-V] Mark C_MV as a move instruction
ClosedPublic

Authored by arichardson on Aug 25 2020, 3:45 AM.

Diff Detail

Event Timeline

arichardson created this revision.Aug 25 2020, 3:45 AM
arichardson requested review of this revision.Aug 25 2020, 3:46 AM
luismarques accepted this revision.Aug 25 2020, 4:43 AM

LGTM.

llvm/lib/Target/RISCV/RISCVInstrInfoC.td
522

I was wondering if it made sense to have both isMoveReg and isAsCheapAsMove, since the former should imply the latter. Unfortunately, that doesn't seem to be handled automatically, so I guess we need both.

This revision is now accepted and ready to land.Aug 25 2020, 4:43 AM
This revision was automatically updated to reflect the committed changes.