[PATCH] D10679: Internalize: internalize comdat members as a group, and drop comdat on such members.

Rafael Ávila de Espíndola rafael.espindola at gmail.com
Fri Jul 10 13:53:16 PDT 2015


rafael added a subscriber: rafael.

================
Comment at: lib/Transforms/IPO/Internalize.cpp:256
@@ -216,3 +255,3 @@
 
-  return Changed;
+  return true;
 }
----------------
Any reason to drop the tracking of Changed? I don't think it makes a difference given how this pass is used, but we should at least comment on why it is not done.


http://reviews.llvm.org/D10679







More information about the llvm-commits mailing list