[PATCH] D91639: Add documentation illustrating use of IgnoreUnlessSpelledInSource
Stephen Kelly via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Nov 20 02:35:02 PST 2020
steveire added inline comments.
================
Comment at: clang/docs/LibASTMatchersReference.html:91-94
+This mode is hard to use correctly and
+requires more development iteration because it means
+that the user must write AST Matchers to explicitly traverse or ignore nodes
+which are not visible.</p>
----------------
aaron.ballman wrote:
> I was trying to reword this to avoid making a judgmental statement that the default mode is hard to use correctly -- I'd strike this part of the paragraph. WDYT?
Oops, that's what I was trying to do, but accidentally left the previous stuff behind. Gone now.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D91639/new/
https://reviews.llvm.org/D91639
More information about the cfe-commits
mailing list