[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