[PATCH] Implement ADRP CSE for global symbols

Quentin Colombet qcolombet at apple.com
Thu May 8 10:01:28 PDT 2014

Hi Jiangning,

Thanks for the update.

I’ll give a shot at the patch as it is and get back to you with performance numbers.



Comment at: lib/Transforms/Scalar/GlobalMerge.cpp:215
@@ +214,3 @@
+    if (EnableGlobalMergeOnExternal && IsExternal) {
+      // If the alignment is not a power of 2, round up to the next power of 2.
Any reason why we do not want to set an alignment for the internal too?


More information about the llvm-commits mailing list