[llvm] r303096 - Add support for handling ifuncs to GlobalValue::getBaseObject
Teresa Johnson via llvm-commits
llvm-commits at lists.llvm.org
Mon May 15 11:28:29 PDT 2017
Author: tejohnson
Date: Mon May 15 13:28:29 2017
New Revision: 303096
URL: http://llvm.org/viewvc/llvm-project?rev=303096&view=rev
Log:
Add support for handling ifuncs to GlobalValue::getBaseObject
Summary:
All GlobalIndirectSymbol types (not just GlobalAlias) should return
their base object.
Without this patch LTO would warn "Unable to determine comdat of
alias!" for an ifunc.
Reviewers: pcc
Subscribers: mehdi_amini, inglorion, llvm-commits
Differential Revision: https://reviews.llvm.org/D33202
Added:
llvm/trunk/test/LTO/Resolution/X86/ifunc.ll
Modified:
llvm/trunk/lib/IR/Globals.cpp
llvm/trunk/test/Object/X86/nm-ir.ll
Modified: llvm/trunk/lib/IR/Globals.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/IR/Globals.cpp?rev=303096&r1=303095&r2=303096&view=diff
==============================================================================
--- llvm/trunk/lib/IR/Globals.cpp (original)
+++ llvm/trunk/lib/IR/Globals.cpp Mon May 15 13:28:29 2017
@@ -255,7 +255,7 @@ bool GlobalValue::canIncreaseAlignment()
const GlobalObject *GlobalValue::getBaseObject() const {
if (auto *GO = dyn_cast<GlobalObject>(this))
return GO;
- if (auto *GA = dyn_cast<GlobalAlias>(this))
+ if (auto *GA = dyn_cast<GlobalIndirectSymbol>(this))
return GA->getBaseObject();
return nullptr;
}
Added: llvm/trunk/test/LTO/Resolution/X86/ifunc.ll
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/LTO/Resolution/X86/ifunc.ll?rev=303096&view=auto
==============================================================================
--- llvm/trunk/test/LTO/Resolution/X86/ifunc.ll (added)
+++ llvm/trunk/test/LTO/Resolution/X86/ifunc.ll Mon May 15 13:28:29 2017
@@ -0,0 +1,15 @@
+; RUN: opt -module-summary -o %t.bc %s
+; RUN: llvm-lto2 run %t.bc -r %t.bc,foo,pl -o %t2
+; RUN: llvm-nm %t2.0 | FileCheck %s
+; CHECK: T foo
+; CHECK: t foo_ifunc
+
+target datalayout = "e-m:e-i64:64-f80:128-n8:16:32:64-S128"
+target triple = "x86_64-unknown-linux-gnu"
+
+ at foo = ifunc i32 (i32), i64 ()* @foo_ifunc
+
+define internal i64 @foo_ifunc() {
+entry:
+ ret i64 0
+}
Modified: llvm/trunk/test/Object/X86/nm-ir.ll
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Object/X86/nm-ir.ll?rev=303096&r1=303095&r2=303096&view=diff
==============================================================================
--- llvm/trunk/test/Object/X86/nm-ir.ll (original)
+++ llvm/trunk/test/Object/X86/nm-ir.ll Mon May 15 13:28:29 2017
@@ -12,7 +12,7 @@
; CHECK-NEXT: C g3
; CHECK-NOT: g4
; CHECK-NEXT: T global_asm_sym
-; CHECK-NEXT: D ifunc_f1
+; CHECK-NEXT: T ifunc_f1
; CHECK-NEXT: t local_asm_sym
; CHECK-NEXT: U undef_asm_sy
More information about the llvm-commits
mailing list