[PATCH] D19144: Handle TemplateArgument in DynTypedNode comparison operations.

Samuel Benzaquen via cfe-commits cfe-commits at lists.llvm.org
Tue Apr 19 10:11:19 PDT 2016


sbenza added a comment.

> > We can proceed with this change if you want, but it is not required anymore. I don't know whether we need the extra complexity of `TemplateArgumentLess`.

> 

> 

> If this patch is not going to help with performance, I'm happy to abandon it.


It might help in the cases where you are doing hasAncestor with TemplateArgument bound nodes, but that is a small corner case and I don't think we need to optimize for it.


http://reviews.llvm.org/D19144





More information about the cfe-commits mailing list