[PATCH] D24492: [LTO] Switch to the new resolution-based API.

[Davide Italiano via llvm-commits]

davide added a comment.

Back to more interesting business, the two asm inline tests that are failing are also failing with the gold plugin, so that might inidcate a bug in lib/LTO. Here's a synthetic testcase.

  $ /usr/bin/ld.gold -shared -m elf_x86_64 -o /home/davide/work/llvm/build/test/tools/gold/X86/Output/module_asm.ll.tmp2 -plugin /home/davide/work/llvm/build/./lib/LLVMgold.so /home/davide/work/llvm/build/test/tools/gold/X86/Output/module_asm.ll.tmp.o
  ld.gold: ../lib/Linker/IRMover.cpp:1240: llvm::Error (anonymous namespace)::IRLinker::run(): Assertion `!GV->isDeclaration()' failed.



  ; RUN: llvm-as %s -o %t.o
  ; RUN: %gold -shared -m elf_x86_64 -o %t2 -plugin %llvmshlibdir/LLVMgold.so %t.o
  
  target datalayout = "e-m:e-i64:64-f80:128-n8:16:32:64-S128"
  target triple = "x86_64-unknown-linux-gnu"
  
  module asm ".weak patatino"
  module asm ".equ patatino, foo"
  
  declare void @patatino()
  
  define void @foo() {
    ret void
  }
  
  define void @_start() {
    call void @patatino()
    ret void
  }


https://reviews.llvm.org/D24492