[cfe-commits] r97897 - in /cfe/trunk: lib/Driver/Tools.cpp test/Driver/clang-g-opts.c

[Rafael Espindola]

Author: rafael
Date: Sat Mar  6 22:46:18 2010
New Revision: 97897

URL: http://llvm.org/viewvc/llvm-project?rev=97897&view=rev
Log:
Don't produce debug info when given -g. Fixes PR6529.

Added:
    cfe/trunk/test/Driver/clang-g-opts.c
Modified:
    cfe/trunk/lib/Driver/Tools.cpp

Modified: cfe/trunk/lib/Driver/Tools.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Driver/Tools.cpp?rev=97897&r1=97896&r2=97897&view=diff
==============================================================================
--- cfe/trunk/lib/Driver/Tools.cpp (original)
+++ cfe/trunk/lib/Driver/Tools.cpp Sat Mar  6 22:46:18 2010
@@ -929,7 +929,9 @@
 
   // Special case debug options to only pass -g to clang. This is
   // wrong.
-  if (Args.hasArg(options::OPT_g_Group))
+  Args.ClaimAllArgs(options::OPT_g_Group);
+  Arg *Garg = Args.getLastArg(options::OPT_g_Group);
+  if (Garg && Garg != Args.getLastArg(options::OPT_g0))
     CmdArgs.push_back("-g");
 
   Args.AddLastArg(CmdArgs, options::OPT_nostdinc);

Added: cfe/trunk/test/Driver/clang-g-opts.c
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Driver/clang-g-opts.c?rev=97897&view=auto
==============================================================================
--- cfe/trunk/test/Driver/clang-g-opts.c (added)
+++ cfe/trunk/test/Driver/clang-g-opts.c Sat Mar  6 22:46:18 2010
@@ -0,0 +1,5 @@
+// RUN: %clang -S -v -o %t %s        2>&1 | not grep -w -- -g
+// RUN: %clang -S -v -o %t %s -g     2>&1 | grep -w -- -g
+// RUN: %clang -S -v -o %t %s -g0    2>&1 | not grep -w -- -g
+// RUN: %clang -S -v -o %t %s -g -g0 2>&1 | not grep -w -- -g
+// RUN: %clang -S -v -o %t %s -g0 -g 2>&1 | grep -w -- -g