6502/llvm-6502
1
0
Fork 0
mirror of https://github.com/c64scene-ar/llvm-6502.git synced 2025-01-12 17:32:19 +00:00
Code Issues Projects Releases Wiki Activity

This patch fixes LTO's RecordStreamer so that it records symbols in the MCExpr

Browse Source 
part of an asm .symver directive as being used. This prevents referenced
functions from being internalized and deleted.

Without the patch to LTOModule.cpp, the test case will produce the error:

LLVM ERROR: A @@ version cannot be undefined.



git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@205221 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Tom Roeder 2014-03-31 16:59:13 +00:00
parent bd5fac585e
commit 5ca1ddf668
2 changed files with 17 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
lib/LTO/LTOModule.cpp
View File

@ -660,6 +660,7 @@ namespace {
void EmitAssignment(MCSymbol *Symbol, const MCExpr *Value) override {
// FIXME: should we handle aliases?
markDefined(*Symbol);
AddValueSymbols(Value);
}
bool EmitSymbolAttribute(MCSymbol *Symbol,
MCSymbolAttr Attribute) override {

16
test/LTO/symver-asm.ll Normal file
View File

@ -0,0 +1,16 @@
; RUN: llvm-as < %s >%t1
; RUN: llvm-lto -o %t2 %t1
; RUN: llvm-nm %t2 | FileCheck %s
target triple = "x86_64-unknown-linux-gnu"
module asm ".symver io_cancel_0_4,io_cancel@@LIBAIO_0.4"
; Even without -exported-symbol, io_cancel_0_4 should be noticed by LTOModule's
; RecordStreamer, so it shouldn't get eliminated. However, the object file will
; contain the aliased symver as well as the original.
define i32 @io_cancel_0_4() {
; CHECK: io_cancel@@LIBAIO_0.4
; CHECK: io_cancel_0_4
ret i32 0
}