Re: [PATCH] D18127: Remove compile time PreserveName in favor of a runtime cc1 -discard-value-names option

[Chandler Carruth via cfe-commits]

chandlerc added a comment.

Change these tests to use %clang_cc1 rather than add all the messy regex to 
handle arguments losing names?


================
Comment at: lib/Driver/Tools.cpp:3715
@@ -3714,1 +3714,3 @@
   CmdArgs.push_back("-disable-llvm-verifier");
+  // Disable LLVM value names in -asserts builds.
+  CmdArgs.push_back("-discard-value-names");
----------------
s/Disable/Discard/


http://reviews.llvm.org/D18127



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits