[PATCH] D95290: [RISCV] Add isel patterns to optimize slli.uw patterns without Zba extension.

[Jessica Clarke via Phabricator via llvm-commits]

jrtc27 added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVInstrInfo.td:336
+// Return an immediate subtracted from 32.
+def ThirtyTwoMinusImm : SDNodeXForm<imm, [{
+  return CurDAG->getTargetConstant(32 - N->getZExtValue(), SDLoc(N),
----------------
craig.topper wrote:
> jrtc27 wrote:
> > jrtc27 wrote:
> > > This is just NegImm and ImmSub32 composed?
> > (also a shorter name that doesn't require spelling 32 in words would be ImmSubFrom32)
> SDNodeXForms can't be composed can they?
Hmm, you can in the patterns but seems there's no way to declare an alias for that.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D95290/new/

https://reviews.llvm.org/D95290