[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