Change default traversal in AST Matchers to ignore invisible nodes

[steveire]

Update

[steveire]

Update

[aaron.ballman]

LGTM aside from some documentation nits.

clang/docs/ReleaseNotes.rst

255 ↗

(On Diff #263552)

is -> are

Also, I think we typically use single spacing after the full stop in the release notes.

257–259 ↗

(On Diff #263552)

We also have release notes for clang-tools-extra that should be updated for the changes to clang-query.

[steveire]

Touch

[steveire]

Touch

[steveire]

Rebase

clang/include/clang/AST/ASTContext.h

private:
   /// address spaces (e.g. OpenCL/CUDA)
   bool AddrSpaceMapMangling;
 
   const TargetInfo *Target = nullptr;
   const TargetInfo *AuxTarget = nullptr;
   clang::PrintingPolicy PrintingPolicy;
   std::unique_ptr<interp::Context> InterpContext;
 
-  ast_type_traits::TraversalKind Traversal = ast_type_traits::TK_AsIs;
+  ast_type_traits::TraversalKind Traversal =
+      ast_type_traits::TK_IgnoreUnlessSpelledInSource;
 
 public:
   ast_type_traits::TraversalKind getTraversalKind() const { return Traversal; }
   void setTraversalKind(ast_type_traits::TraversalKind TK) { Traversal = TK; }
 
   const Expr *traverseIgnored(const Expr *E) const;
   Expr *traverseIgnored(Expr *E) const;
   ast_type_traits::DynTypedNode