[PATCH] D126661: [MachineVerifier] Fix crash on early clobbered subreg operands.

[Daniil Fukalov via Phabricator via llvm-commits]

dfukalov added inline comments.


================
Comment at: llvm/lib/CodeGen/MachineVerifier.cpp:2266
+    // somewhere is performed in visitMachineFunctionAfter()
+    if (((SubRangeCheck || MO->getSubReg() == 0) && (VNI->def != DefIdx)) ||
+        !SlotIndex::isSameInstr(VNI->def, DefIdx) ||
----------------
foad wrote:
> Don't need the last set of parens on this line.
Formally yes, but without the parens I got
```
warning: '&&' within '||' [-Wlogical-op-parentheses]
note: place parentheses around the '&&' expression to silence this warning
```


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D126661