a.sidorin added a comment.

> `E->getFoundDecl().getDecl()` can be null when a member expression does not 
> involve lookup. (Note, it may involve a lookup in case of a using directive 
> which refers to a member function in a base class template.)

Yes, a pretty weird example. Unfortunately, they are pretty common for XTU.

> I hoped that this patch could be accepted without tests, since it is very 
> small and the changes are pretty straight forward, so I thought it can be 
> easily verified by a review.

Yes, this is the reason why I have accepted it initially.
I think we can accept this patch without a test but I'd like to get a somebody 
else's review. @xazax.hun, @szepet, could you take a look?
@martong Could you include the reason why this patch lacks tests into the 
commit message (including description of the situation where the code fails)?


Repository:
  rC Clang

https://reviews.llvm.org/D46019



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to