[PATCH] D22374: [analyzer] Copy and move constructors - ExprEngine extended for "almost trivial" copy and move constructors

Balogh, Ádám via cfe-commits cfe-commits at lists.llvm.org
Wed Oct 26 07:39:11 PDT 2016


baloghadamsoftware added a comment.

In https://reviews.llvm.org/D22374#575579, @NoQ wrote:

> Ping!~ Did my idea sound completely wrong to you? :)
>
> Does https://reviews.llvm.org/D25660 depend on this patch? And/or did you find another workaround?
>
> upd.: I also thought that this deserves more code comments - the purpose of the new mechanism should be more obvious by reading the code.


Sorry, since it turned out that https://reviews.llvm.org/D25660 does not depend on this patch I downprioritized it.

By some mistake I missed the notification about your patch-over-patch. It seems OK to me.


https://reviews.llvm.org/D22374





More information about the cfe-commits mailing list