[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