[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