[PATCH] D82439: [ARM] Rearrange SizeReduction when using -Oz

Dave Green via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 29 01:34:36 PDT 2020


dmgreen accepted this revision.
dmgreen added a comment.
This revision is now accepted and ready to land.

LGTM



================
Comment at: llvm/lib/Target/ARM/ARMTargetMachine.cpp:521
   if (getOptLevel() != CodeGenOpt::None) {
     // in v8, IfConversion depends on Thumb instruction widths
+    // Always run Thumb2SizeReduction before IfConversion
----------------
Feel free to make this into more of a sentence too, with Capitalization and a full stop.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D82439





More information about the llvm-commits mailing list