[PATCH] D82226: Add Metadata to Transformer tooling

Andy Soffer via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jun 29 08:03:41 PDT 2020


asoffer added a comment.

I'm not against a map, but I don't see the need for it right now. We don't have multiple stages that write to the same AtomicChange in any tool I'm aware of. Given that these are immutable after construction, I think just the Any is simpler. So long as we're willing to break things later (Does LLVM have a policy on API compatibility?) I think it's better to start with the simpler thing.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D82226





More information about the cfe-commits mailing list