Author: bruno Date: Mon May 2 15:20:49 2016 New Revision: 268297 URL: http://llvm.org/viewvc/llvm-project?rev=268297&view=rev Log: Revert "[Driver] Quote clang full version in dwarf producer when invoking cc1as"
This reverts commit r264813 / 6484b95d634f53dd929c75265ef3c4decf397584. While using it in the shell is fine, this a problem when cc1as is invoked directly by the driver because single quoting the clang full version makes cc1as write out the version with the quotes in the final binary. If the user wants to copy-n-pastable output, it could use either -### or CC_PRINT_OPTIONS=1 clang -v ... Modified: cfe/trunk/lib/Driver/Tools.cpp cfe/trunk/test/Misc/cc1as-asm.s Modified: cfe/trunk/lib/Driver/Tools.cpp URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Driver/Tools.cpp?rev=268297&r1=268296&r2=268297&view=diff ============================================================================== --- cfe/trunk/lib/Driver/Tools.cpp (original) +++ cfe/trunk/lib/Driver/Tools.cpp Mon May 2 15:20:49 2016 @@ -6251,8 +6251,7 @@ void ClangAs::ConstructJob(Compilation & // Set the AT_producer to the clang version when using the integrated // assembler on assembly source files. CmdArgs.push_back("-dwarf-debug-producer"); - std::string QuotedClangVersion("'" + getClangFullVersion() + "'"); - CmdArgs.push_back(Args.MakeArgString(QuotedClangVersion)); + CmdArgs.push_back(Args.MakeArgString(getClangFullVersion())); // And pass along -I options Args.AddAllArgs(CmdArgs, options::OPT_I); Modified: cfe/trunk/test/Misc/cc1as-asm.s URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Misc/cc1as-asm.s?rev=268297&r1=268296&r2=268297&view=diff ============================================================================== --- cfe/trunk/test/Misc/cc1as-asm.s (original) +++ cfe/trunk/test/Misc/cc1as-asm.s Mon May 2 15:20:49 2016 @@ -1,5 +1,3 @@ // Run cc1as asm output path just to make sure it works // REQUIRES: x86-registered-target // RUN: %clang -cc1as -triple x86_64-apple-macosx10.10.0 -filetype asm %s -o /dev/null -// Test that cc1as is able to consume a quoted clang full version -// RUN: %clang -cc1as -triple x86_64-apple-macosx10.10.0 -dwarf-debug-producer 'clang dummy version' -filetype asm %s -o /dev/null _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits