[PATCH] D82557: [MSP430][NFC] Remove redundant condition

Anton Korobeynikov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 25 13:05:23 PDT 2020


asl requested changes to this revision.
asl added inline comments.
This revision now requires changes to proceed.


================
Comment at: llvm/lib/Target/MSP430/MSP430FrameLowering.cpp:91
     // the two.
     //NumBytes -= mergeSPUpdates(MBB, MBBI, true);
     // If there is an ADD16ri or SUB16ri of SP immediately after this
----------------
Please do not add commented-out code. Thanks!


================
Comment at: llvm/lib/Target/MSP430/MSP430FrameLowering.cpp:95
     // mergeSPUpdatesDown(MBB, MBBI, &NumBytes);
+    //if (NumBytes) {
 
----------------
Ditto


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D82557





More information about the llvm-commits mailing list