[PATCH] D79330: [Analyzer][VLASizeChecker] Check for VLA size overflow.

Gabor Marton via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu May 7 08:29:51 PDT 2020


martong added a comment.

Would it be possible to split this patch into two?

1. The refactoring part where you move code out from checkPreStmt to checkVLA. This should be an NFC.
2. Handling of the overflow.

Would be much cleaner I guess.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D79330/new/

https://reviews.llvm.org/D79330





More information about the cfe-commits mailing list