[PATCH] D30738: Don't internalize llvm GV's with InternalizeLinkedSymbols
Peter Collingbourne via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Mar 9 15:35:48 PST 2017
pcc added a comment.
The fix can be simplified to
if (P.first().startswith("llvm."))
continue;
That is two lines of code. Even if we simplified the internalize pass in the same way, we should not add this much complexity just to avoid duplicating two lines.
Repository:
rL LLVM
https://reviews.llvm.org/D30738
More information about the llvm-commits
mailing list