[PATCH] D61335: [LibTooling] Add support to Transformer for composing rules as an ordered choice.
Ilya Biryukov via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri May 3 07:08:45 PDT 2019
ilya-biryukov added a comment.
> As for naming, agreed, but does that concern drop away once we have only a single RewriteRule definition?
Sure, that won't be an issue.
The use-cases make sense, thanks for the examples. Could you add one or two to the code? Would probably make sense to express them in code as matchers, rather than explaining what we're trying to do in a natural language.
Let's proceed with making `RewriteRule` the vocabulary type that we use for transformations like this if we both agree that's a good idea.
There are obviously multiple ways to tackle this, which one you had in mind?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D61335/new/
https://reviews.llvm.org/D61335
More information about the cfe-commits
mailing list