Teach the internalize pass to skip dllexported symbols because they could be

referenced in a way that even the linker does not see.

Differential Revision: http://llvm-reviews.chandlerc.com/D2280



git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@196300 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Yunzhong Gao 2013-12-03 18:05:14 +00:00
parent df9c9da884
commit 822a02fb4c
2 changed files with 13 additions and 0 deletions

View File

@ -115,6 +115,10 @@ static bool shouldInternalize(const GlobalValue &GV,
if (GV.hasAvailableExternallyLinkage())
return false;
// Assume that dllexported symbols are referenced elsewhere
if (GV.hasDLLExportLinkage())
return false;
// Already has internal linkage
if (GV.hasLocalLinkage())
return false;

View File

@ -48,3 +48,12 @@ define void @foo() {
define available_externally void @bar() {
ret void
}
; ALL: define dllexport void @export_foo() {
; FOO_AND_J: define dllexport void @export_foo() {
; FOO_AND_BAR: define dllexport void @export_foo() {
; FOO_J_AND_BAR: define dllexport void @export_foo() {
define dllexport void @export_foo() {
ret void
}