[PATCH] D20276: [MBP] Reduce code size by running tail merging in MBP

Haicheng Wu via llvm-commits llvm-commits at lists.llvm.org
Tue May 24 11:46:54 PDT 2016


haicheng added inline comments.

================
Comment at: lib/CodeGen/MachineBlockPlacement.cpp:1496
@@ +1495,3 @@
+    }
+  }
+
----------------
deadalnix wrote:
> Worth looping maybe ?
I did some research on spec2006 about looping.  Nine benchmarks can further remove instructions.  gcc, omnetpp, and perlbench can remove 1000+ instructions.  83.5% MFs across spec2006 only need one iteration, 15.3% MFs need two iterations.  The largest iteration number is five which is needed by 0.03% MFs.

It seems promising to pursue, but I think I will do the looping in the next patch after this one stays in the tree for a while.   


Repository:
  rL LLVM

http://reviews.llvm.org/D20276





More information about the llvm-commits mailing list