[PATCH] D41278: [MachineCombiner] Improve debug output (NFC)

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 29 14:01:11 PST 2017


RKSimon added inline comments.


================
Comment at: lib/CodeGen/MachineCombiner.cpp:139
+#ifndef NDEBUG
+  const TargetSubtargetInfo &STI = MF->getSubtarget();
+#endif
----------------
Is STI used in this function? I couldn't see it.


================
Comment at: lib/CodeGen/MachineCombiner.cpp:578
   MinInstr = nullptr;
-  OptSize = MF.getFunction().optForSize();
+  OptSize = MF->getFunction().optForSize();
 
----------------
If we're storing MF do we need to keep MRI and OptSize ?


https://reviews.llvm.org/D41278





More information about the llvm-commits mailing list