X86: repair export compatibility with MinGW/cygwin

Both MinGW and cygwin (i686) construct export directives without the global
leader prefix.  This is mostly due to the fact that they use GNU ld which does
not correctly handle the export directive.  This apparently has been been broken
for a while.  However, this was recently reported as being broken by
mingwandroid and diorcety of the msys2 project.

Remove the global leader prefix if targeting MinGW or cygwin, otherwise, retain
the global leader prefix.  Add an explicit test for cygwin's behaviour of export
directives.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@207926 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Saleem Abdulrasool 2014-05-04 00:03:48 +00:00
parent 6f0d4e358f
commit f3b2ed7498
2 changed files with 47 additions and 37 deletions

View File

@ -553,13 +553,18 @@ emitNonLazySymbolPointer(MCStreamer &OutStreamer, MCSymbol *StubLabel,
void X86AsmPrinter::GenerateExportDirective(const MCSymbol *Sym, bool IsData) {
SmallString<128> Directive;
raw_svector_ostream OS(Directive);
StringRef Name = Sym->getName();
if (Subtarget->isTargetKnownWindowsMSVC())
OS << " /EXPORT:";
else
OS << " -export:";
OS << Sym->getName();
if ((Subtarget->isTargetWindowsGNU() || Subtarget->isTargetWindowsCygwin()) &&
(Name[0] == getDataLayout().getGlobalPrefix()))
Name = Name.drop_front();
OS << Name;
if (IsData) {
if (Subtarget->isTargetKnownWindowsMSVC())

View File

@ -1,5 +1,9 @@
; RUN: llc -mtriple i386-pc-win32 < %s | FileCheck -check-prefix=CHECK -check-prefix=WIN32 %s
; RUN: llc -mtriple i386-pc-mingw32 < %s | FileCheck -check-prefix=CHECK -check-prefix=MINGW %s
; RUN: llc -mtriple i386-pc-win32 < %s \
; RUN: | FileCheck -check-prefix CHECK -check-prefix CHECK-CL %s
; RUN: llc -mtriple i386-pc-mingw32 < %s \
; RUN: | FileCheck -check-prefix CHECK -check-prefix CHECK-GCC %s
; RUN: llc -mtriple i686-pc-cygwin %s -o - \
; RUN: | FileCheck -check-prefix CHECK -check-prefix CHECK-GCC %s
; CHECK: .text
@ -89,37 +93,38 @@ define weak_odr dllexport void @weak1() {
; CHECK: .section .drectve
; WIN32: /EXPORT:_Var1,DATA
; WIN32: /EXPORT:_Var2,DATA
; WIN32: /EXPORT:_Var3,DATA
; WIN32: /EXPORT:_WeakVar1,DATA
; WIN32: /EXPORT:_WeakVar2,DATA
; WIN32: /EXPORT:_f1
; WIN32: /EXPORT:_f2
; WIN32: /EXPORT:_stdfun@0
; WIN32: /EXPORT:@fastfun@0
; WIN32: /EXPORT:_thisfun
; WIN32: /EXPORT:_lnk1
; WIN32: /EXPORT:_lnk2
; WIN32: /EXPORT:_weak1
; WIN32: /EXPORT:_alias
; WIN32: /EXPORT:_alias2
; WIN32: /EXPORT:_alias3
; WIN32: /EXPORT:_weak_alias
; MINGW: -export:_Var1,data
; MINGW: -export:_Var2,data
; MINGW: -export:_Var3,data
; MINGW: -export:_WeakVar1,data
; MINGW: -export:_WeakVar2,data
; MINGW: -export:_f1
; MINGW: -export:_f2
; MINGW: -export:_stdfun@0
; MINGW: -export:@fastfun@0
; MINGW: -export:_thisfun
; MINGW: -export:_lnk1
; MINGW: -export:_lnk2
; MINGW: -export:_weak1
; MINGW: -export:_alias
; MINGW: -export:_alias2
; MINGW: -export:_alias3
; MINGW: -export:_weak_alias
; CHECK-CL: /EXPORT:_Var1,DATA
; CHECK-CL: /EXPORT:_Var2,DATA
; CHECK-CL: /EXPORT:_Var3,DATA
; CHECK-CL: /EXPORT:_WeakVar1,DATA
; CHECK-CL: /EXPORT:_WeakVar2,DATA
; CHECK-CL: /EXPORT:_f1
; CHECK-CL: /EXPORT:_f2
; CHECK-CL: /EXPORT:_stdfun@0
; CHECK-CL: /EXPORT:@fastfun@0
; CHECK-CL: /EXPORT:_thisfun
; CHECK-CL: /EXPORT:_lnk1
; CHECK-CL: /EXPORT:_lnk2
; CHECK-CL: /EXPORT:_weak1
; CHECK-CL: /EXPORT:_alias
; CHECK-CL: /EXPORT:_alias2
; CHECK-CL: /EXPORT:_alias3
; CHECK-CL: /EXPORT:_weak_alias
; CHECK-GCC: -export:Var1,data
; CHECK-GCC: -export:Var2,data
; CHECK-GCC: -export:Var3,data
; CHECK-GCC: -export:WeakVar1,data
; CHECK-GCC: -export:WeakVar2,data
; CHECK-GCC: -export:f1
; CHECK-GCC: -export:f2
; CHECK-GCC: -export:stdfun@0
; CHECK-GCC: -export:@fastfun@0
; CHECK-GCC: -export:thisfun
; CHECK-GCC: -export:lnk1
; CHECK-GCC: -export:lnk2
; CHECK-GCC: -export:weak1
; CHECK-GCC: -export:alias
; CHECK-GCC: -export:alias2
; CHECK-GCC: -export:alias3
; CHECK-GCC: -export:weak_alias