[PATCH] D13249: Divide TraverseInitListExpr to a different function for each form.

Manuel Klimek via cfe-commits cfe-commits at lists.llvm.org
Tue Sep 29 21:03:18 PDT 2015


I think it's worth figuring out when this is called with the semantic or
syntactic version and why this can't lead to double visitation. Then add a
comment while you're changing the method so the next person doesn't have to
figure it all out :)

On Wed, Sep 30, 2015 at 12:15 AM Angel Garcia <angelgarcia at google.com>
wrote:

> angelgarcia added a comment.
>
> Yes, it breaks a few tests:
>
> FAIL: Clang :: Analysis/operator-calls.cpp (598 of 8596)
>
> FAIL: Clang :: Analysis/misc-ps-region-store.cpp (599 of 8596)
>
> FAIL: Clang :: Analysis/array-struct-region.c (602 of 8596)
>
>
> http://reviews.llvm.org/D13249
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20150930/e4bf588c/attachment.html>


More information about the cfe-commits mailing list