r260344 - libclang: Enable skip-parsed-bodies on win32.

NAKAMURA Takumi via cfe-commits cfe-commits at lists.llvm.org
Tue Feb 9 17:29:58 PST 2016


Author: chapuni
Date: Tue Feb  9 19:29:57 2016
New Revision: 260344

URL: http://llvm.org/viewvc/llvm-project?rev=260344&view=rev
Log:
libclang: Enable skip-parsed-bodies on win32.

I guess it would be working since Rafael's r187619.

Modified:
    cfe/trunk/test/Index/skip-parsed-bodies/compile_commands.json
    cfe/trunk/tools/libclang/Indexing.cpp

Modified: cfe/trunk/test/Index/skip-parsed-bodies/compile_commands.json
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Index/skip-parsed-bodies/compile_commands.json?rev=260344&r1=260343&r2=260344&view=diff
==============================================================================
--- cfe/trunk/test/Index/skip-parsed-bodies/compile_commands.json (original)
+++ cfe/trunk/test/Index/skip-parsed-bodies/compile_commands.json Tue Feb  9 19:29:57 2016
@@ -16,7 +16,6 @@
 }
 ]
 
-// XFAIL: mingw32,win32,windows-gnu
 // RUN: c-index-test -index-compile-db %s | FileCheck %s
 
 // CHECK:      [startedTranslationUnit]

Modified: cfe/trunk/tools/libclang/Indexing.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/tools/libclang/Indexing.cpp?rev=260344&r1=260343&r2=260344&view=diff
==============================================================================
--- cfe/trunk/tools/libclang/Indexing.cpp (original)
+++ cfe/trunk/tools/libclang/Indexing.cpp Tue Feb  9 19:29:57 2016
@@ -45,26 +45,6 @@ namespace {
 // Skip Parsed Bodies
 //===----------------------------------------------------------------------===//
 
-#ifdef LLVM_ON_WIN32
-
-// FIXME: On windows it is disabled since current implementation depends on
-// file inodes.
-
-class SessionSkipBodyData { };
-
-class TUSkipBodyControl {
-public:
-  TUSkipBodyControl(SessionSkipBodyData &sessionData,
-                    PPConditionalDirectiveRecord &ppRec,
-                    Preprocessor &pp) { }
-  bool isParsed(SourceLocation Loc, FileID FID, const FileEntry *FE) {
-    return false;
-  }
-  void finished() { }
-};
-
-#else
-
 /// \brief A "region" in source code identified by the file/offset of the
 /// preprocessor conditional directive that it belongs to.
 /// Multiple, non-consecutive ranges can be parts of the same region.
@@ -238,8 +218,6 @@ private:
   }
 };
 
-#endif
-
 //===----------------------------------------------------------------------===//
 // IndexPPCallbacks
 //===----------------------------------------------------------------------===//




More information about the cfe-commits mailing list