[PATCH] D158499: [analyzer] Compute FAM dynamic size

Ding Fei via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Aug 22 23:49:22 PDT 2023


danix800 added a comment.

> I suspected that the wrong cast modeling and how we infer what value ranges are calculated is susceptible to such APSInt signedness issues, but I haven't seen a case in the wild for extents. Thus, I didn't think of fixing it either. But yes, we should.

I think cast is not the problem, size-type inconsistent is. For `ArrayBound`, the assumption should be done with correct types to avoid `-Wsign-compare` issue.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D158499



More information about the cfe-commits mailing list