diff options
author | David L. Jones <dlj@google.com> | 2019-08-15 04:10:11 +0000 |
---|---|---|
committer | David L. Jones <dlj@google.com> | 2019-08-15 04:10:11 +0000 |
commit | 4ed5521cad7961566e29185b760ddad402e3ba7a (patch) | |
tree | d098dde606b2ac99f5b9a67622509b7964bf4924 /llvm/lib/CodeGen/AsmPrinter/CodeViewDebug.cpp | |
parent | 10e3fa28d0df4d17130fab08a2c65e6cf7d2c01b (diff) | |
download | llvm-4ed5521cad7961566e29185b760ddad402e3ba7a.zip llvm-4ed5521cad7961566e29185b760ddad402e3ba7a.tar.gz llvm-4ed5521cad7961566e29185b760ddad402e3ba7a.tar.bz2 |
[Tooling] Add a hack to work around issues with matcher binding in r368681.
The change in r368681 contains a (probably unintentional) behavioral change for
rewrite rules with a single matcher. Previously, the single matcher would not
need to be bound (`joinCaseMatchers` returned it directly), even though a final
DynTypeMatcher was created and bound by `buildMatcher`. With the new change, a
single matcher will be bound, in addition to the final binding (which is now in
`buildMatchers`, but happens roughly at the same point in the overall flow).
This patch simply duplicates the "final matcher" trick: it creates an extra
DynTypedMatcher for each rewrite rule case matcher, and unconditionally makes it
bindable. This is probably not the right long-term fix, but it does allow
existing code to continue to work with this interface.
Subscribers: cfe-commits, gribozavr, ymandel
Tags: #clang
Differential Revision: https://reviews.llvm.org/D66273
llvm-svn: 368958
Diffstat (limited to 'llvm/lib/CodeGen/AsmPrinter/CodeViewDebug.cpp')
0 files changed, 0 insertions, 0 deletions