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

Richard Smith - zygoloid via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Oct 1 14:54:13 PDT 2018


rsmith added a comment.

We're going to try evaluating the array size anyway (in `isArraySizeVLA`). It would be much better to produce the overflow warnings as part of that evaluation rather than adding an extra evaluation step to produce them.


https://reviews.llvm.org/D52750





More information about the cfe-commits mailing list