[PATCH] D147266: [AArch64] Sink operands to allow for bitselect instructions

[Pranav Kant via Phabricator via cfe-commits]

pranavk added inline comments.


================
Comment at: llvm/lib/Target/AArch64/AArch64ISelLowering.cpp:14361-14363
+          for (unsigned Idx = 0; Idx < MainAnd->getNumOperands(); Idx++) {
+            if (MainAnd->getOperand(Idx) != IA) {
+              Ops.push_back(&MainAnd->getOperandUse(Idx));
----------------
pranavk wrote:
> dmgreen wrote:
> > I think this can avoid the loop if we just use
> > `Ops.push_back(&MainAnd->getOperandUse(MainAnd->getOperand(0) == IA ? 1 : 0));`
> llvm/lib/Target/AArch64/AArch64ISelLowering.cpp
Done. this was left as part of my final refactoring. Uploaded new patch.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D147266