[clang-diff] Simplify mapping
ClosedPublic

Authored by johannes on Aug 1 2017, 1:54 PM.

Details

Summary

Until we find a decent heuristic on how to choose between multiple
identical trees, there is no point in supporting multiple mappings.

This also enables matching of nodes with parents of different types,
because there are many instances where this is appropriate. For
example for and foreach statements; functions in the global or
other namespaces.

Diff Detail

Repository
rL LLVM
johannes created this revision.Aug 1 2017, 1:54 PM
johannes updated this revision to Diff 109306.Aug 2 2017, 2:57 AM

parent changed

johannes updated this revision to Diff 109422.Aug 2 2017, 1:40 PM
johannes edited the summary of this revision. (Show Details)
johannes updated this revision to Diff 109504.Aug 3 2017, 2:42 AM

merge parent changes

arphaman added inline comments.Aug 9 2017, 4:38 AM
include/clang/Tooling/ASTDiff/ASTDiff.h
114 ↗(On Diff #109504)

It doesn't look like this field is used in this patch, please remove it or move it to a different patch.

johannes updated this revision to Diff 110554.Aug 10 2017, 4:19 AM

remove unused

This revision is now accepted and ready to land.Aug 10 2017, 10:15 AM
This revision was automatically updated to reflect the committed changes.