[PATCH] D79072: [Analyzer][VLASizeChecker] Check VLA size in typedef and sizeof.

Balázs Kéri via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu May 7 23:57:35 PDT 2020


balazske added a comment.

This change is relatively small and the refactoring like part (introduction of `checkVLA` if I think correct?) is connected to its use. The other change is add of a new function (callback). This is probably small enough to go into one change and we can see why the new function `checkVLA` is needed. So my vote is to not split this change.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D79072





More information about the cfe-commits mailing list