[PATCH] D30738: Don't internalize llvm GV's with InternalizeLinkedSymbols

Teresa Johnson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 9 15:29:42 PST 2017


tejohnson added a comment.

In https://reviews.llvm.org/D30738#697024, @pcc wrote:

> Is the callback really necessary? I don't see anything fundamentally wrong with the first version of the patch.


I don't think we should be duplicating logic that is already available elsewhere.


Repository:
  rL LLVM

https://reviews.llvm.org/D30738





More information about the llvm-commits mailing list