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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits