[PATCH] D113917: Add infrastructure to support matcher names.

Wed Dec 15 05:40:34 PST 2021

aaron.ballman added a comment.

In D113917#3193374 <https://reviews.llvm.org/D113917#3193374>, @ymandel wrote:

> Aaron - agreed on the points about `StringRef` vs `std::string`. But, before I make that change, what did you think of moving to a more general method `getMatcherSpec` that returns an object? That object will provide the name (for now) and other data in the future (like the enum we discussed in the RFC thread)?

I think that's a reasonable approach!

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D113917