[PATCH] D101005: [AArch64][GlobalISel] Simplify out of range rotate amount.

[Amara Emerson via Phabricator via llvm-commits]

aemerson added inline comments.


================
Comment at: llvm/lib/CodeGen/GlobalISel/CombinerHelper.cpp:3924
+  auto Bits = Builder.buildConstant(AmtTy, Bitsize);
+  ConstantFoldingMIRBuilder CFMIB(MI);
+  Amt = CFMIB.buildURem(AmtTy, MI.getOperand(2).getReg(), Bits).getReg(0);
----------------
arsenm wrote:
> aemerson wrote:
> > arsenm wrote:
> > > I think one off MIRBuilders should never be used
> > What do you suggest here instead?
> The combiner helper should have a universal builder
The universal builder is a CSE one, and I need a constant folding one here. Maybe CSE should do it too?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D101005