<p dir="ltr">LGTM thanks!</p>
<div class="gmail_quote">On Jul 26, 2013 1:17 PM, "Hans Wennborg" <<a href="mailto:hans@chromium.org">hans@chromium.org</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
  Please take another look.<br>
<br>
Hi chandlerc, rnk,<br>
<br>
<a href="http://llvm-reviews.chandlerc.com/D1215" target="_blank">http://llvm-reviews.chandlerc.com/D1215</a><br>
<br>
CHANGE SINCE LAST DIFF<br>
  <a href="http://llvm-reviews.chandlerc.com/D1215?vs=3017&id=3040#toc" target="_blank">http://llvm-reviews.chandlerc.com/D1215?vs=3017&id=3040#toc</a><br>
<br>
Files:<br>
  include/clang/Driver/CLCompatOptions.td<br>
  include/clang/Driver/Driver.h<br>
  include/clang/Driver/Makefile<br>
  include/clang/Driver/Options.h<br>
  include/clang/Driver/Options.td<br>
  lib/Driver/Driver.cpp<br>
  test/Driver/cl.c<br>
<br>
Index: include/clang/Driver/CLCompatOptions.td<br>
===================================================================<br>
--- /dev/null<br>
+++ include/clang/Driver/CLCompatOptions.td<br>
@@ -0,0 +1,21 @@<br>
+//===--- CLCompatOptions.td - Options for clang-cl ------------------------===//<br>
+//<br>
+//                     The LLVM Compiler Infrastructure<br>
+//<br>
+// This file is distributed under the University of Illinois Open Source<br>
+// License. See LICENSE.TXT for details.<br>
+//<br>
+//===----------------------------------------------------------------------===//<br>
+//<br>
+//  This file defines the options accepted by clang-cl.<br>
+//<br>
+//===----------------------------------------------------------------------===//<br>
+<br>
+def cl_Group : OptionGroup<"<clang-cl options>">,<br>
+    HelpText<"CL.EXE COMPATIBILITY OPTIONS">;<br>
+<br>
+class CLFlag<string name> : Option<["/", "-"], name, KIND_FLAG>,<br>
+    Group<cl_Group>, Flags<[CLOption]>;<br>
+<br>
+def _QUESTION : CLFlag<"?">, Alias<help>, HelpText<"Display available options">;<br>
+def cl_help : CLFlag<"help">, Alias<help>, HelpText<"Display available options">;<br>
Index: include/clang/Driver/Driver.h<br>
===================================================================<br>
--- include/clang/Driver/Driver.h<br>
+++ include/clang/Driver/Driver.h<br>
@@ -396,6 +396,10 @@<br>
<br>
   /// @}<br>
<br>
+  /// \brief Get bitmasks for which option flags to include and exclude based on<br>
+  /// the driver mode.<br>
+  std::pair<unsigned, unsigned> getIncludeExcludeOptionFlagMasks() const;<br>
+<br>
 public:<br>
   /// GetReleaseVersion - Parse (([0-9]+)(.([0-9]+)(.([0-9]+)?))?)? and<br>
   /// return the grouped values as integers. Numbers which are not<br>
Index: include/clang/Driver/Makefile<br>
===================================================================<br>
--- include/clang/Driver/Makefile<br>
+++ include/clang/Driver/Makefile<br>
@@ -5,7 +5,7 @@<br>
<br>
 include $(CLANG_LEVEL)/Makefile<br>
<br>
-$(ObjDir)/Options.inc.tmp : Options.td CC1Options.td $(LLVM_TBLGEN) $(ObjDir)/.dir<br>
+$(ObjDir)/Options.inc.tmp : Options.td CC1Options.td CLCompatOptions.td $(LLVM_TBLGEN) $(ObjDir)/.dir<br>
        $(Echo) "Building Clang Driver Option tables with tblgen"<br>
        $(Verb) $(LLVMTableGen) -gen-opt-parser-defs -o $(call SYSPATH, $@) $<<br>
<br>
Index: include/clang/Driver/Options.h<br>
===================================================================<br>
--- include/clang/Driver/Options.h<br>
+++ include/clang/Driver/Options.h<br>
@@ -27,8 +27,9 @@<br>
   LinkerInput = (1 << 5),<br>
   NoArgumentUnused = (1 << 6),<br>
   Unsupported = (1 << 7),<br>
-  CC1Option = (1 << 8),<br>
-  NoDriverOption = (1 << 9)<br>
+  CLOption = (1 << 8),<br>
+  CC1Option = (1 << 9),<br>
+  NoDriverOption = (1 << 10)<br>
 };<br>
<br>
 enum ID {<br>
Index: include/clang/Driver/Options.td<br>
===================================================================<br>
--- include/clang/Driver/Options.td<br>
+++ include/clang/Driver/Options.td<br>
@@ -33,6 +33,10 @@<br>
 // lines that use it.<br>
 def Unsupported : OptionFlag;<br>
<br>
+// CLOption - This is a cl.exe compatibility option. Options with this flag<br>
+// are made available when the driver is running in CL compatibility mode.<br>
+def CLOption : OptionFlag;<br>
+<br>
 // CC1Option - This option should be accepted by clang -cc1.<br>
 def CC1Option : OptionFlag;<br>
<br>
@@ -97,6 +101,7 @@<br>
 //   _ => __<br>
 //   - => _<br>
 //   # => _HASH<br>
+//   ? => _QUESTION<br>
 //   , => _COMMA<br>
 //   = => _EQ<br>
 //   C++ => CXX<br>
@@ -1314,4 +1319,6 @@<br>
 def Z_reserved_lib_cckext : Flag<["-"], "Z-reserved-lib-cckext">,<br>
     Flags<[LinkerInput, NoArgumentUnused, Unsupported]>, Group<reserved_lib_Group>;<br>
<br>
+include "CLCompatOptions.td"<br>
+<br>
 include "CC1Options.td"<br>
Index: lib/Driver/Driver.cpp<br>
===================================================================<br>
--- lib/Driver/Driver.cpp<br>
+++ lib/Driver/Driver.cpp<br>
@@ -20,6 +20,7 @@<br>
 #include "clang/Driver/ToolChain.h"<br>
 #include "llvm/ADT/ArrayRef.h"<br>
 #include "llvm/ADT/OwningPtr.h"<br>
+#include "llvm/ADT/STLExtras.h"<br>
 #include "llvm/ADT/StringSet.h"<br>
 #include "llvm/ADT/StringSwitch.h"<br>
 #include "llvm/Option/Arg.h"<br>
@@ -108,9 +109,17 @@<br>
<br>
 InputArgList *Driver::ParseArgStrings(ArrayRef<const char *> ArgList) {<br>
   llvm::PrettyStackTraceString CrashInfo("Command line argument parsing");<br>
+<br>
+  unsigned IncludedFlagsBitmask;<br>
+  unsigned ExcludedFlagsBitmask;<br>
+  llvm::tie(IncludedFlagsBitmask, ExcludedFlagsBitmask) =<br>
+    getIncludeExcludeOptionFlagMasks();<br>
+<br>
   unsigned MissingArgIndex, MissingArgCount;<br>
   InputArgList *Args = getOpts().ParseArgs(ArgList.begin(), ArgList.end(),<br>
-                                           MissingArgIndex, MissingArgCount);<br>
+                                           MissingArgIndex, MissingArgCount,<br>
+                                           IncludedFlagsBitmask,<br>
+                                           ExcludedFlagsBitmask);<br>
<br>
   // Check for missing argument error.<br>
   if (MissingArgCount)<br>
@@ -607,9 +616,17 @@<br>
 }<br>
<br>
 void Driver::PrintHelp(bool ShowHidden) const {<br>
-  getOpts().PrintHelp(<br>
-      llvm::outs(), Name.c_str(), DriverTitle.c_str(), /*Include*/ 0,<br>
-      /*Exclude*/ options::NoDriverOption | (ShowHidden ? 0 : HelpHidden));<br>
+  unsigned IncludedFlagsBitmask;<br>
+  unsigned ExcludedFlagsBitmask;<br>
+  llvm::tie(IncludedFlagsBitmask, ExcludedFlagsBitmask) =<br>
+    getIncludeExcludeOptionFlagMasks();<br>
+<br>
+  ExcludedFlagsBitmask |= options::NoDriverOption;<br>
+  if (!ShowHidden)<br>
+    ExcludedFlagsBitmask |= HelpHidden;<br>
+<br>
+  getOpts().PrintHelp(llvm::outs(), Name.c_str(), DriverTitle.c_str(),<br>
+                      IncludedFlagsBitmask, ExcludedFlagsBitmask);<br>
 }<br>
<br>
 void Driver::PrintVersion(const Compilation &C, raw_ostream &OS) const {<br>
@@ -1856,3 +1873,18 @@<br>
   HadExtra = true;<br>
   return true;<br>
 }<br>
+<br>
+std::pair<unsigned, unsigned> Driver::getIncludeExcludeOptionFlagMasks() const {<br>
+  unsigned IncludedFlagsBitmask = 0;<br>
+  unsigned ExcludedFlagsBitmask = 0;<br>
+<br>
+  if (Mode == CLMode) {<br>
+    // Only allow CL options.<br>
+    // FIXME: Also allow "core" Clang options.<br>
+    IncludedFlagsBitmask = options::CLOption;<br>
+  } else {<br>
+    ExcludedFlagsBitmask |= options::CLOption;<br>
+  }<br>
+<br>
+  return std::make_pair(IncludedFlagsBitmask, ExcludedFlagsBitmask);<br>
+}<br>
Index: test/Driver/cl.c<br>
===================================================================<br>
--- test/Driver/cl.c<br>
+++ test/Driver/cl.c<br>
@@ -1,3 +1,21 @@<br>
-// RUN: %clang_cl -fsyntax-only -c %s<br>
+// Check that clang-cl options are not available by default.<br>
+// RUN: %clang -help | FileCheck %s -check-prefix=DEFAULT<br>
+// DEFAULT-NOT: CL.EXE COMPATIBILITY OPTIONS<br>
+// DEFAULT-NOT: {{/[?]}}<br>
+// DEFAULT-NOT: /help<br>
+// RUN: not %clang /?<br>
+// RUN: not %clang -?<br>
+// RUN: not %clang /help<br>
<br>
-void f();<br>
+// Check that /? and /help are available as clang-cl options.<br>
+// RUN: %clang_cl /? | FileCheck %s -check-prefix=CL<br>
+// RUN: %clang_cl /help | FileCheck %s -check-prefix=CL<br>
+// RUN: %clang_cl -help | FileCheck %s -check-prefix=CL<br>
+// CL: CL.EXE COMPATIBILITY OPTIONS<br>
+// CL: {{/[?]}}<br>
+// CL: /help<br>
+<br>
+// Options which are not "core" clang options nor cl.exe compatible options<br>
+// are not available in clang-cl.<br>
+// DEFAULT: -fapple-kext<br>
+// CL-NOT: -fapple-kext<br>
<br>_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
<br></blockquote></div>