[PATCH] D59329: [LibTooling] Add NodeId, a strong type for AST-matcher node identifiers.

Ilya Biryukov via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Mar 29 07:05:47 PDT 2019


ilya-biryukov added a comment.

Could you provide the rationale for having `NodeID` vs just using strings for the binds?
Is this just a more type-safe way to do the same thing or is that actually required to solve a particular problem?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D59329/new/

https://reviews.llvm.org/D59329





More information about the cfe-commits mailing list