[cfe-commits] r86686 - in /cfe/trunk: TODO.txt tools/clang-cc/clang-cc.cpp

[Daniel Dunbar]

Author: ddunbar
Date: Tue Nov 10 10:23:44 2009
New Revision: 86686

URL: http://llvm.org/viewvc/llvm-project?rev=86686&view=rev
Log:
Cleanup some clang-cc FIXMEs

Modified:
    cfe/trunk/TODO.txt
    cfe/trunk/tools/clang-cc/clang-cc.cpp

Modified: cfe/trunk/TODO.txt
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/TODO.txt?rev=86686&r1=86685&r2=86686&view=diff

==============================================================================
--- cfe/trunk/TODO.txt (original)
+++ cfe/trunk/TODO.txt Tue Nov 10 10:23:44 2009
@@ -66,3 +66,12 @@
   - If a class member is defined out-of-line but isn't in the class declaration (and there are no close matches!), provide the option to add an in-class declaration.
   - Fix-it hints for the inclusion of headers when needed for particular features (e.g., <typeinfo> for typeid)
   - Change "foo.bar" to "foo->bar" when "foo" is a pointer.
+
+//===---------------------------------------------------------------------===//
+
+Options to support:
+   -Wfatal-errors
+   -ftabstop=width
+   -fpreprocessed mode.
+   -nostdinc++
+   -imultilib

Modified: cfe/trunk/tools/clang-cc/clang-cc.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/tools/clang-cc/clang-cc.cpp?rev=86686&r1=86685&r2=86686&view=diff

==============================================================================
--- cfe/trunk/tools/clang-cc/clang-cc.cpp (original)
+++ cfe/trunk/tools/clang-cc/clang-cc.cpp Tue Nov 10 10:23:44 2009
@@ -14,13 +14,6 @@
 //   clang [options] file1 file2 - Read these files.
 //
 //===----------------------------------------------------------------------===//
-//
-// TODO: Options to support:
-//
-//   -Wfatal-errors
-//   -ftabstop=width
-//
-//===----------------------------------------------------------------------===//
 
 #include "clang/Frontend/AnalysisConsumer.h"
 #include "clang/Frontend/ASTConsumers.h"
@@ -446,7 +439,6 @@
 
 
 static void InitializeLangOptions(LangOptions &Options, LangKind LK){
-  // FIXME: implement -fpreprocessed mode.
   bool NoPreprocess = false;
 
   switch (LK) {
@@ -641,7 +633,6 @@
 MainFileName("main-file-name",
              llvm::cl::desc("Main file name to use for debug info"));
 
-// FIXME: Also add an "-fno-access-control" option.
 static llvm::cl::opt<bool>
 AccessControl("faccess-control",
               llvm::cl::desc("Enable C++ access control"));
@@ -963,10 +954,6 @@
 // This tool exports a large number of command line options to control how the
 // preprocessor searches for header files.  At root, however, the Preprocessor
 // object takes a very simple interface: a list of directories to search for
-//
-// FIXME: -nostdinc++
-// FIXME: -imultilib
-//
 
 static llvm::cl::opt<bool>
 nostdinc("nostdinc", llvm::cl::desc("Disable standard #include directories"));
@@ -1341,8 +1328,8 @@
   Opts.Features.clear();
   for (llvm::StringMap<bool>::const_iterator it = Features.begin(),
          ie = Features.end(); it != ie; ++it) {
-    // FIXME: If we are completely confident that we have the right
-    // set, we only need to pass the minuses.
+    // FIXME: If we are completely confident that we have the right set, we only
+    // need to pass the minuses.
     std::string Name(it->second ? "+" : "-");
     Name += it->first();
     Opts.Features.push_back(Name);
@@ -1420,7 +1407,6 @@
 DependencyTargets("MT",
          llvm::cl::desc("Specify target for dependency"));
 
-// FIXME: Implement feature
 static llvm::cl::opt<bool>
 PhonyDependencyTarget("MP",
             llvm::cl::desc("Create phony target for each dependency "