serge-sans-paille added inline comments.
================
Comment at: clang/test/SemaCXX/array-bounds.cpp:240
- ((char*)foo)[sizeof(foo)] = '\0'; // expected-warning {{array index 32768
is past the end of the array (which contains 32768 elements)}}
+ ((char*)foo)[sizeof(foo)] = '\0'; // expected-warning {{array index 32768
is past the end of the array (which contains 4096 elements)}}
----------------
I find this new message quite confusing, because the array index is given in
terms of char elements, while the array size is given in terms of double
elements. I actually find the original message more consistent to that respect.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D135920/new/
https://reviews.llvm.org/D135920
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
