mirror of
https://github.com/RPCS3/llvm-mirror.git
synced 2024-11-23 11:13:28 +01:00
[COFF] Don't try to add quotes to already quoted linker directives
If a linker directive is already quoted, don't try to quote it again, otherwise it creates a mess. This pops up in places like: #pragma comment(linker,"\"/foo bar'\"") Differential Revision: http://reviews.llvm.org/D6792 llvm-svn: 224998
This commit is contained in:
parent
be9ae58d93
commit
a000cb8396
@ -932,7 +932,7 @@ emitModuleFlags(MCStreamer &Streamer,
|
||||
StringRef Op = MDOption->getString();
|
||||
// Lead with a space for consistency with our dllexport implementation.
|
||||
std::string Escaped(" ");
|
||||
if (Op.find(" ") != StringRef::npos) {
|
||||
if (!Op.startswith("\"") && (Op.find(" ") != StringRef::npos)) {
|
||||
// The PE-COFF spec says args with spaces must be quoted. It doesn't say
|
||||
// how to escape quotes, but it probably uses this algorithm:
|
||||
// http://msdn.microsoft.com/en-us/library/17w5ykft(v=vs.85).aspx
|
||||
|
@ -1,6 +1,6 @@
|
||||
; RUN: llc -O0 -mtriple=i386-pc-win32 -filetype=asm -o - %s | FileCheck %s
|
||||
|
||||
!0 = !{i32 6, !"Linker Options", !{!{!"/DEFAULTLIB:msvcrt.lib"}, !{!"/DEFAULTLIB:msvcrt.lib", !"/DEFAULTLIB:secur32.lib"}, !{!"/DEFAULTLIB:C:\5Cpath to\5Casan_rt.lib"}, !{!"/with spaces"}}}
|
||||
!0 = !{i32 6, !"Linker Options", !{!{!"/DEFAULTLIB:msvcrt.lib"}, !{!"/DEFAULTLIB:msvcrt.lib", !"/DEFAULTLIB:secur32.lib"}, !{!"/DEFAULTLIB:C:\5Cpath to\5Casan_rt.lib"}, !{!"/with spaces"}, !{!"\22/quoted spaces\22"}}}
|
||||
|
||||
!llvm.module.flags = !{ !0 }
|
||||
|
||||
@ -14,4 +14,5 @@ define dllexport void @foo() {
|
||||
; CHECK: .ascii " /DEFAULTLIB:secur32.lib"
|
||||
; CHECK: .ascii " \"/DEFAULTLIB:C:\\path to\\asan_rt.lib\""
|
||||
; CHECK: .ascii " \"/with spaces\""
|
||||
; CHECK: .ascii " \"/quoted spaces\""
|
||||
; CHECK: .ascii " /EXPORT:_foo"
|
||||
|
Loading…
Reference in New Issue
Block a user