[PATCH] D48341: [clang-doc] Adding a second reduce pass

Julie Hockett via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jul 2 14:15:34 PDT 2018


juliehockett added inline comments.


================
Comment at: clang-tools-extra/clang-doc/tool/ClangDocMain.cpp:249
+    if (EnclosingScope.get().RefType == doc::InfoType::IT_function)
+      continue;
+    
----------------
ioeric wrote:
> Are symbols declared in functions indexed by the tool? If not, this should probably assert or at least emit a warning message.
They are currently, but I'm planning to eliminate that in a future patch and will add the appropriate assertion then.


https://reviews.llvm.org/D48341





More information about the cfe-commits mailing list