gribozavr added inline comments.
================
Comment at: test/Analysis/cxx-uninitialized-object.cpp:1155
+void __vector_size__LongTest() {
+ VectorSizeLong v;
+}
----------------
gribozavr wrote:
> gribozavr wrote:
> > Could you also add tests where the vector member is being subscripted? for
> > example, `v.x[0] = 0;`
> Should there be a warning about the member being uninitialized?
Just to be clear -- if there should be a warning, I don't think this patch has
to necessarily fix everything in the checker and static analyzer core to
support vector types, the fix for the crash is valuable by itself. However, if
there should be a warning, this test should have a fixme.
Repository:
rC Clang
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D61246/new/
https://reviews.llvm.org/D61246
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
