From: Daniel Dunbar <daniel@zuster.org>
Date: Tue, 10 Nov 2009 16:23:44 +0000 (+0000)
Subject: Cleanup some clang-cc FIXMEs
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=9af869510b89fb8d99fca68be18e32f91954da19;p=clang

Cleanup some clang-cc FIXMEs

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@86686 91177308-0d34-0410-b5e6-96231b3b80d8
---

diff --git a/TODO.txt b/TODO.txt
index 522dcd37b3..7ceb0da15b 100644
--- a/TODO.txt
+++ b/TODO.txt
@@ -66,3 +66,12 @@ More ideas for code modification hints:
   - 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
diff --git a/tools/clang-cc/clang-cc.cpp b/tools/clang-cc/clang-cc.cpp
index ae1b32f71f..861e20e47f 100644
--- a/tools/clang-cc/clang-cc.cpp
+++ b/tools/clang-cc/clang-cc.cpp
@@ -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 InitializeObjCOptions(LangOptions &Options) {
 
 
 static void InitializeLangOptions(LangOptions &Options, LangKind LK){
-  // FIXME: implement -fpreprocessed mode.
   bool NoPreprocess = false;
 
   switch (LK) {
@@ -641,7 +633,6 @@ static llvm::cl::opt<std::string>
 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 @@ RelocatablePCH("relocatable-pch",
 // 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 @@ static void InitializeCompileOptions(CompileOptions &Opts,
   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 @@ static llvm::cl::list<std::string>
 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 "