r204755 - Use an option alias to implement -gmlt

[David Blaikie]

Author: dblaikie
Date: Tue Mar 25 15:42:27 2014
New Revision: 204755

URL: http://llvm.org/viewvc/llvm-project?rev=204755&view=rev
Log:
Use an option alias to implement -gmlt

Review feedback from Reid Kleckner on r203603.

Modified:
    cfe/trunk/include/clang/Driver/Options.td
    cfe/trunk/lib/Driver/Tools.cpp

Modified: cfe/trunk/include/clang/Driver/Options.td
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Driver/Options.td?rev=204755&r1=204754&r2=204755&view=diff
==============================================================================
--- cfe/trunk/include/clang/Driver/Options.td (original)
+++ cfe/trunk/include/clang/Driver/Options.td Tue Mar 25 15:42:27 2014
@@ -898,8 +898,7 @@ def g_Flag : Flag<["-"], "g">, Group<g_G
   HelpText<"Generate source level debug information">, Flags<[CC1Option]>;
 def gline_tables_only : Flag<["-"], "gline-tables-only">, Group<g_Group>,
   HelpText<"Emit debug line number tables only">, Flags<[CC1Option]>;
-def gmlt : Flag<["-"], "gmlt">, Group<g_Group>,
-  HelpText<"Emit debug line number tables only">, Flags<[DriverOption,HelpHidden]>;
+def gmlt : Flag<["-"], "gmlt">, Alias<gline_tables_only>;
 def g0 : Flag<["-"], "g0">, Group<g_Group>;
 def g1 : Flag<["-"], "g1">, Group<g_Group>;
 def g2 : Flag<["-"], "g2">, Group<g_Group>;

Modified: cfe/trunk/lib/Driver/Tools.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Driver/Tools.cpp?rev=204755&r1=204754&r2=204755&view=diff
==============================================================================
--- cfe/trunk/lib/Driver/Tools.cpp (original)
+++ cfe/trunk/lib/Driver/Tools.cpp Tue Mar 25 15:42:27 2014
@@ -2695,8 +2695,7 @@ void Clang::ConstructJob(Compilation &C,
   // are preserved, all other debug options are substituted with "-g".
   Args.ClaimAllArgs(options::OPT_g_Group);
   if (Arg *A = Args.getLastArg(options::OPT_g_Group)) {
-    if (A->getOption().matches(options::OPT_gline_tables_only) ||
-        A->getOption().matches(options::OPT_gmlt)) {
+    if (A->getOption().matches(options::OPT_gline_tables_only)) {
       // FIXME: we should support specifying dwarf version with
       // -gline-tables-only.
       CmdArgs.push_back("-gline-tables-only");