[PATCH] D52750: [Diagnostics] Check for integer overflow in array size expressions

Nicolas Lesser via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Oct 12 08:39:11 PDT 2018


Rakete1111 added a comment.

In https://reviews.llvm.org/D52750#1263466, @xbolva00 wrote:

> In https://reviews.llvm.org/D52750#1263461, @Rakete1111 wrote:
>
> > This doesn't produce a warning in C++11 and up.
>
>
> But see Richard's comment: https://reviews.llvm.org/D52750#125175 so I am not sure :/


I guess I can see why it makes sense to suppress the warning in those cases. Sorry.


https://reviews.llvm.org/D52750





More information about the cfe-commits mailing list