diff --git a/test/Index/Inputs/get-cursor-includes-1.h b/test/Index/Inputs/get-cursor-includes-1.h new file mode 100644 index 0000000000000000000000000000000000000000..48439d2ae265ee6fbbe9f2353d7f0c52ca4294f0 --- /dev/null +++ b/test/Index/Inputs/get-cursor-includes-1.h @@ -0,0 +1,6 @@ +#ifndef GET_CURSOR_INCLUDES_1_H +#define GET_CURSOR_INCLUDES_1_H + +extern int blah; + +#endif // GET_CURSOR_INCLUDES_1_H diff --git a/test/Index/Inputs/get-cursor-includes-2.h b/test/Index/Inputs/get-cursor-includes-2.h new file mode 100644 index 0000000000000000000000000000000000000000..cf95c1828be7b31aa9f338918d10d7ecbf98b9e4 --- /dev/null +++ b/test/Index/Inputs/get-cursor-includes-2.h @@ -0,0 +1,2 @@ +#include "get-cursor-includes-1.h" +#include "get-cursor-includes-1.h" diff --git a/test/Index/get-cursor-includes.c b/test/Index/get-cursor-includes.c new file mode 100644 index 0000000000000000000000000000000000000000..68d64f144673c7089f0136af63b99743ec49d3f7 --- /dev/null +++ b/test/Index/get-cursor-includes.c @@ -0,0 +1,7 @@ +#include "get-cursor-includes-2.h" +#include "get-cursor-includes-2.h" + +// RUN: c-index-test -write-pch %t.h.pch -I%S/Inputs -Xclang -detailed-preprocessing-record %S/Inputs/get-cursor-includes-2.h +// RUN: c-index-test -cursor-at=%S/Inputs/get-cursor-includes-2.h:1:5 -I%S/Inputs -include %t.h %s | FileCheck %s + +// CHECK: inclusion directive=get-cursor-includes-1.h diff --git a/tools/libclang/CIndex.cpp b/tools/libclang/CIndex.cpp index 38b58e0068eefae2920903be5a30fa6424c3edc4..bd39925dadef259973ce3f6ec921b2e2397d7c78 100644 --- a/tools/libclang/CIndex.cpp +++ b/tools/libclang/CIndex.cpp @@ -410,7 +410,20 @@ CursorVisitor::getPreprocessedEntities() { = *AU->getPreprocessor().getPreprocessingRecord(); bool OnlyLocalDecls - = !AU->isMainFileAST() && AU->getOnlyLocalDecls(); + = !AU->isMainFileAST() && AU->getOnlyLocalDecls(); + + if (OnlyLocalDecls && RegionOfInterest.isValid()) { + // If we would only look at local declarations but we have a region of + // interest, check whether that region of interest is in the main file. + // If not, we should traverse all declarations. + // FIXME: My kingdom for a proper binary search approach to finding + // cursors! + std::pair<FileID, unsigned> Location + = AU->getSourceManager().getDecomposedInstantiationLoc( + RegionOfInterest.getBegin()); + if (Location.first != AU->getSourceManager().getMainFileID()) + OnlyLocalDecls = false; + } PreprocessingRecord::iterator StartEntity, EndEntity; if (OnlyLocalDecls) {