[PATCH] D30588: [LTO] Add module asm uses to the llvm.compiler.used before merging

Teresa Johnson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 3 13:06:53 PST 2017


tejohnson created this revision.
Herald added a subscriber: mehdi_amini.

In LTO mode we were adding undefined module asm uses to the
llvm.compiler.used eventually, but not early enough. The referenced
internal symbol was simply not being linked into the combined module
since it didn't appear referenced.

Fixes PR25679.

Depends on https://reviews.llvm.org/D30585


https://reviews.llvm.org/D30588

Files:
  include/llvm/LTO/LTOBackend.h
  include/llvm/LTO/legacy/LTOCodeGenerator.h
  include/llvm/LTO/legacy/LTOModule.h
  lib/LTO/LTO.cpp
  lib/LTO/LTOBackend.cpp
  lib/LTO/LTOCodeGenerator.cpp
  lib/LTO/LTOModule.cpp
  test/LTO/X86/symver-asm.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D30588.90523.patch
Type: text/x-patch
Size: 8441 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170303/7fceeb57/attachment.bin>


More information about the llvm-commits mailing list