[PATCH] D94894: [RISCV] Add way to mark CompressPats that should only be used for compressing.
Craig Topper via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jan 18 09:33:10 PST 2021
craig.topper added a comment.
In D94894#2504716 <https://reviews.llvm.org/D94894#2504716>, @jrtc27 wrote:
> My instinct is this should be a priority to match InstAlias; is there ever any overlap?
After this patch, uncompressing is a 1:1 mapping. In the compress direction there are multiple patterns that map to the same sequence. There are also multiple patterns that start from ADD for example. Overlaps seem to be handled by Register classes like GPRNoX0 and by immediate predicates. If a priority is needed, I believe it would be needed in the compress direction, not the uncompress direction.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D94894/new/
https://reviews.llvm.org/D94894
More information about the llvm-commits
mailing list