RPCS3/llvm-mirror
1
0
Fork 0
mirror of https://github.com/RPCS3/llvm-mirror.git synced 2024-12-14 15:19:33 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Revert r185872 - "Stop emitting weak symbols into the "coal" sections"

Browse Source 
This patch broke `make check-asan` on Mac, causing ld warnings like the following one:

ld: warning: direct access in __GLOBAL__I_a to global weak symbol
___asan_mapping_scale means the weak symbol cannot be overridden at
runtime. This was likely caused by different translation units being
compiled with different visibility settings.

The resulting test binaries crashed with incorrect ASan warnings.

llvm-svn: 185923
This commit is contained in:
Alexander Potapenko 2013-07-09 10:00:16 +00:00
parent 7f5f52a614
commit 9c55668f5b
3 changed files with 10 additions and 83 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
lib/CodeGen/TargetLoweringObjectFileImpl.cpp
View File

@ -524,14 +524,14 @@ const MCSection *TargetLoweringObjectFileMachO::
SelectSectionForGlobal(const GlobalValue *GV, SectionKind Kind,
Mangler *Mang, const TargetMachine &TM) const {
if (Kind.isText())
return TextSection;
// If this is weak/linkonce, put this in a read only or data section depending
// on whether or not it's writable.
return GV->isWeakForLinker() ? TextCoalSection : TextSection;
// If this is weak/linkonce, put this in a coalescable section, either in text
// or data depending on if it is writable.
if (GV->isWeakForLinker()) {
if (Kind.isReadOnly())
return ReadOnlySection;
return DataSection;
return ConstTextCoalSection;
return DataCoalSection;
}
// FIXME: Alignment check should be handled by section classifier.

7
test/CodeGen/X86/global-sections.ll
View File

@ -20,7 +20,7 @@
; TODO: linux drops this into .rodata, we drop it into ".gnu.linkonce.r.G2"
; DARWIN: .section __TEXT,__const
; DARWIN: .section __TEXT,__const_coal,coalesced
; DARWIN: _G2:
; DARWIN: .long 42
@ -70,6 +70,7 @@
; LINUX: .weak foo_20_bar
; LINUX: foo_20_bar:
; DARWIN: .section __DATA,__datacoal_nt,coalesced
; DARWIN: .globl "_foo bar"
; DARWIN: .weak_definition "_foo bar"
; DARWIN: "_foo bar":
@ -84,7 +85,7 @@
; LINUX: .byte 1
; LINUX: .size G6, 1
; DARWIN: .section __TEXT,__const
; DARWIN: .section __TEXT,__const_coal,coalesced
; DARWIN: .globl _G6
; DARWIN: .weak_definition _G6
; DARWIN:_G6:
@ -130,7 +131,7 @@
@G10 = weak global [100 x i32] zeroinitializer, align 32 ; <[100 x i32]*> [#uses=0]
; DARWIN: .section __DATA,__data
; DARWIN: .section __DATA,__datacoal_nt,coalesced
; DARWIN: .globl _G10
; DARWIN: .weak_definition _G10
; DARWIN: .align 5

74
test/CodeGen/X86/no-coal-sections.ll
View File

@ -1,74 +0,0 @@
; RUN: llc < %s -mtriple x86_64-apple-macosx10 | FileCheck %s
; <rdar://problem/14265330>
; CHECK: .section __TEXT,__text
; CHECK-NOT: .section __TEXT,__textcoal_nt
; CHECK: .globl __ZN6TrickyIiLi0EEC1Ev
; CHECK: .section __DATA,__data
; CHECK-NOT: .section __DATA,__datacoal_nt
; CHECK: .globl __ZTV6TrickyIiLi0EE
; CHECK: .section __TEXT,__const
; CHECK-NOT: .section __TEXT,__const_coal
; CHECK: .globl __ZTS6TrickyIiLi0EE
; CHECK: .section __DATA,__data
; CHECK-NOT: .section __DATA,__datacoal_nt
; CHECK: .globl __ZTI6TrickyIiLi0EE
%class.Tricky = type { i32 (...)**, %union.anon }
%union.anon = type { i32 }
@_ZTV6TrickyIiLi0EE = linkonce_odr unnamed_addr constant [4 x i8*] [i8* null, i8* bitcast ({ i8*, i8* }* @_ZTI6TrickyIiLi0EE to i8*), i8* bitcast (void (%class.Tricky*)* @_ZN6TrickyIiLi0EED1Ev to i8*), i8* bitcast (void (%class.Tricky*)* @_ZN6TrickyIiLi0EED0Ev to i8*)]
@_ZTVN10__cxxabiv117__class_type_infoE = external global i8*
@_ZTS6TrickyIiLi0EE = linkonce_odr constant [15 x i8] c"6TrickyIiLi0EE\00"
@_ZTI6TrickyIiLi0EE = linkonce_odr unnamed_addr constant { i8*, i8* } { i8* bitcast (i8** getelementptr inbounds (i8** @_ZTVN10__cxxabiv117__class_type_infoE, i64 2) to i8*), i8* getelementptr inbounds ([15 x i8]* @_ZTS6TrickyIiLi0EE, i32 0, i32 0) }
; Function Attrs: nounwind ssp uwtable
define i32 @main() {
entry:
%ok = alloca %class.Tricky, align 8
call void @_ZN6TrickyIiLi0EEC1Ev(%class.Tricky* %ok)
ret i32 0
}
; Function Attrs: inlinehint nounwind ssp uwtable
define linkonce_odr void @_ZN6TrickyIiLi0EEC1Ev(%class.Tricky* nocapture %this) unnamed_addr align 2 {
entry:
tail call void @_ZN6TrickyIiLi0EEC2Ev(%class.Tricky* %this)
ret void
}
; Function Attrs: nounwind readnone ssp uwtable
define linkonce_odr void @_ZN6TrickyIiLi0EED1Ev(%class.Tricky* nocapture %this) unnamed_addr align 2 {
entry:
ret void
}
; Function Attrs: nounwind readnone ssp uwtable
define linkonce_odr void @_ZN6TrickyIiLi0EED2Ev(%class.Tricky* nocapture %this) unnamed_addr align 2 {
entry:
ret void
}
; Function Attrs: inlinehint nounwind ssp uwtable
define linkonce_odr void @_ZN6TrickyIiLi0EEC2Ev(%class.Tricky* nocapture %this) unnamed_addr align 2 {
entry:
%0 = getelementptr inbounds %class.Tricky* %this, i64 0, i32 0
store i32 (...)** bitcast (i8** getelementptr inbounds ([4 x i8*]* @_ZTV6TrickyIiLi0EE, i64 0, i64 2) to i32 (...)**), i32 (...)*** %0, align 8
ret void
}
; Function Attrs: nounwind ssp uwtable
define linkonce_odr void @_ZN6TrickyIiLi0EED0Ev(%class.Tricky* %this) unnamed_addr align 2 {
invoke.cont:
%0 = bitcast %class.Tricky* %this to i8*
tail call void @_ZdlPv(i8* %0)
ret void
}
declare i32 @__gxx_personality_v0(...)
; Function Attrs: nounwind
declare void @_ZdlPv(i8*)