This is an archive of the discontinued LLVM Phabricator instance.

[RISCV] Map pseudos to their BaseInstr to reduce cases
ClosedPublic

Authored by pcwang-thead on Jun 21 2022, 6:12 AM.

Details

Summary

There are a lot of cases for pseudos of the same instruction, here
we just use existed mapping table to map pseudos to real instructions
to reduce cases.

Diff Detail

Event Timeline

Herald added a project: Restricted Project. · View Herald TranscriptJun 21 2022, 6:12 AM
pcwang-thead requested review of this revision.Jun 21 2022, 6:12 AM
Herald added a project: Restricted Project. · View Herald TranscriptJun 21 2022, 6:12 AM

I think this is not a NFC.

llvm/lib/Target/RISCV/RISCVInsertVSETVLI.cpp
106

I think we missed TU pseudos here.

pcwang-thead retitled this revision from [RISCV][NFC] Map pseudos to their BaseInstr to reduce cases to [RISCV] Map pseudos to their BaseInstr to reduce cases.Jun 23 2022, 5:36 AM
pcwang-thead edited the summary of this revision. (Show Details)

@pcwang-thead sorry for very late review, I think this is very good simplification for the code, could you rebase that?

This revision is now accepted and ready to land.Oct 27 2022, 1:05 AM