[PATCH] D27043: Remove assertion on analysis of rvalue vector

Anna Zaks via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Nov 30 10:52:10 PST 2016

Previous message: [PATCH] D27090: Add LocationContext as a parameter to checkRegionChanges
Next message: r288263 - [analyzer] Construct temporary objects of correct types, destroy them properly.
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

zaks.anna added a comment.

Thank you! The assert definitely needs to be relaxed.

Would be great to add test cases that check if the analyzer models the vector types correctly, not just that we do not crash. (Not a blocker, but would be very useful.)


https://reviews.llvm.org/D27043

Previous message: [PATCH] D27090: Add LocationContext as a parameter to checkRegionChanges
Next message: r288263 - [analyzer] Construct temporary objects of correct types, destroy them properly.
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

More information about the cfe-commits mailing list