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

Reply via email to