[PATCH] D19144: Handle TemplateArgument in DynTypedNode comparison operations.
Alexander Kornienko via cfe-commits
cfe-commits at lists.llvm.org
Mon Apr 18 06:53:48 PDT 2016
alexfh added a comment.
In http://reviews.llvm.org/D19144#402853, @sbenza wrote:
> I think the bug is coming from `memoizedMatchesAncestorOfRecursively`.
> `memoizedMatchesRecursively` has a special case at the top to skip the cache if the node is not sortable. The other function should do that too.
> Although the check is stale also because it is only checking for memoizationData and not whether the node itself works for < and ==.
> Note that adding TemplateArgument to the function is ok, but that won't fix the bug because we still have other nodes that are not comparable.
With this I would be entering "I have no idea what I'm doing" land ;) Might make sense for you to take over the patch, if you know how to fix the issue. WDYT?
More information about the cfe-commits