[PATCH] D101970: [X86FixupLEAs] Transform the sequence LEA/SUB to SUB/SUB

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 12 02:36:16 PDT 2021


RKSimon added inline comments.


================
Comment at: llvm/lib/Target/X86/X86FixupLEAs.cpp:420
+    for (unsigned I = 0, E = CurInst->getNumOperands(); I != E; ++I) {
+      MachineOperand &opnd = CurInst->getOperand(I);
+      if (opnd.isReg() && opnd.getReg() == DestReg) {
----------------
Please can you fix all these case style warnings:
```
MachineOperand &Opnd = CurInst->getOperand(I);
```


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

https://reviews.llvm.org/D101970



More information about the llvm-commits mailing list