[PATCH] D40970: [ThinLTO] Remove unused (?) code from thinLTOInternalizeModule

Mehdi AMINI via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 8 09:16:28 PST 2017


mehdi_amini accepted this revision.
mehdi_amini added a comment.
This revision is now accepted and ready to land.

I wrote this code in April 2016 ( http://reviews.llvm.org/D19103 ), in the very first approach to internalization, which was inferred in the backend and wasn't bullet proof (especially with respect to assembly by the way).
Since then, we moved all decision in the thin-link and this code is indeed (hopefully) dead.

So LGTM! (but it may be a good idea to hear back from Teresa about this)


Repository:
  rL LLVM

https://reviews.llvm.org/D40970





More information about the llvm-commits mailing list