[PATCH] [COFF] Don't try to add quotes to already quoted linker directives
David Majnemer
david.majnemer at gmail.com
Mon Dec 29 02:13:33 PST 2014
================
Comment at: lib/CodeGen/TargetLoweringObjectFileImpl.cpp:989
@@ -988,3 +988,3 @@
std::string Escaped(" ");
- if (Op.find(" ") != StringRef::npos) {
+ if ((Op.find(" ") != StringRef::npos) && (!Op.startswith("\""))) {
// The PE-COFF spec says args with spaces must be quoted. It doesn't say
----------------
This would probably be a little nicer as:
`if (!Op.startswith("\"") && Op.find(" ") != StringRef::npos)`
This way we don't need to hunt for spaces if it starts with a \" sequence.
http://reviews.llvm.org/D6792
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
More information about the llvm-commits
mailing list