[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