[PATCH] D15393: [X86] Order the local stack symbols to improve code size and locality.

Sean Silva via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 9 21:51:24 PST 2016


silvas added inline comments.

================
Comment at: lib/Target/X86/X86FrameLowering.cpp:2697
@@ +2696,3 @@
+    // The density is calculated by doing :
+    //     (double)DensityA = A.ObjectNumUses / A.ObjectSize
+    //     (double)DensityB = B.ObjectNumUses / B.ObjectSize
----------------
Tiny nit: This comment reference "double" but the code doesn't seem to be using floating point anymore.


http://reviews.llvm.org/D15393





More information about the llvm-commits mailing list