[cfe-commits] r115420 - /cfe/trunk/tools/libclang/CIndex.cpp

[Douglas Gregor]

Author: dgregor
Date: Sat Oct  2 15:06:51 2010
New Revision: 115420

URL: http://llvm.org/viewvc/llvm-project?rev=115420&view=rev
Log:
Don't add -fno-spell-checking in libclang if a spell-checking-related argument is already in the command-line arguments

Modified:
    cfe/trunk/tools/libclang/CIndex.cpp

Modified: cfe/trunk/tools/libclang/CIndex.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/tools/libclang/CIndex.cpp?rev=115420&r1=115419&r2=115420&view=diff
==============================================================================
--- cfe/trunk/tools/libclang/CIndex.cpp (original)
+++ cfe/trunk/tools/libclang/CIndex.cpp Sat Oct  2 15:06:51 2010
@@ -2042,9 +2042,17 @@
     // (often very broken) source code, where spell-checking can have a
     // significant negative impact on performance (particularly when 
     // precompiled headers are involved), we disable it by default.
-    // Note that we place this argument early in the list, so that it can be
-    // overridden by the caller with "-fspell-checking".
-    Args.push_back("-fno-spell-checking");
+    // Only do this if we haven't found a spell-checking-related argument.
+    bool FoundSpellCheckingArgument = false;
+    for (int I = 0; I != num_command_line_args; ++I) {
+      if (strcmp(command_line_args[I], "-fno-spell-checking") == 0 ||
+          strcmp(command_line_args[I], "-fspell-checking") == 0) {
+        FoundSpellCheckingArgument = true;
+        break;
+      }
+    }
+    if (!FoundSpellCheckingArgument)
+      Args.push_back("-fno-spell-checking");
     
     Args.insert(Args.end(), command_line_args,
                 command_line_args + num_command_line_args);