[PATCH] D26880: [ThinLTO] Fix crash: transitively referenced global shouldn't be promoted

Teresa Johnson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 1 14:23:27 PST 2016


tejohnson updated this revision to Diff 79983.
tejohnson added a comment.

Complete the fix by removing special casing for static consts from
the promotion logic. Update tests.


https://reviews.llvm.org/D26880

Files:
  lib/Transforms/IPO/FunctionImport.cpp
  lib/Transforms/Utils/FunctionImportUtils.cpp
  test/Linker/funcimport.ll
  test/ThinLTO/X86/Inputs/reference_non_importable.ll
  test/ThinLTO/X86/Inputs/referenced_by_constant.ll
  test/ThinLTO/X86/funcimport.ll
  test/ThinLTO/X86/reference_non_importable.ll
  test/ThinLTO/X86/referenced_by_constant.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D26880.79983.patch
Type: text/x-patch
Size: 8650 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20161201/03cd4ebb/attachment.bin>


More information about the llvm-commits mailing list