diff options
| author | Richard Smith <richard-llvm@metafoo.co.uk> | 2014-05-30 19:44:17 +0000 |
|---|---|---|
| committer | Richard Smith <richard-llvm@metafoo.co.uk> | 2014-05-30 19:44:17 +0000 |
| commit | c2422facc0837a576e322c23395361c76c7f3403 (patch) | |
| tree | 156269c6060da4f58b2331740433ce69dcb8f0e2 | |
| parent | c8fc08c31b1554bc852cc5fc8241c00bd5ac52b1 (diff) | |
| download | bcm5719-llvm-c2422facc0837a576e322c23395361c76c7f3403.tar.gz bcm5719-llvm-c2422facc0837a576e322c23395361c76c7f3403.zip | |
Add a FIXME for a bug Doug pointed out in code review.
llvm-svn: 209915
| -rw-r--r-- | clang/include/clang/Basic/Module.h | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/clang/include/clang/Basic/Module.h b/clang/include/clang/Basic/Module.h index 89fd3886b44..9b66840ba41 100644 --- a/clang/include/clang/Basic/Module.h +++ b/clang/include/clang/Basic/Module.h @@ -425,6 +425,10 @@ public: /// \brief Determine whether the specified module would be visible to /// a lookup at the end of this module. + /// + /// FIXME: This may return incorrect results for (submodules of) the + /// module currently being built, if it's queried before we see all + /// of its imports. bool isModuleVisible(const Module *M) const { if (VisibleModulesCache.empty()) buildVisibleModulesCache(); |
