[PATCH] D99714: [clang][Analyzer] Handle flexible arrays better in ArrayBoundV2 checker.

Gabor Marton via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Apr 1 04:22:49 PDT 2021


martong added a comment.

The tests are really promising! :)



================
Comment at: clang/lib/StaticAnalyzer/Checkers/ArrayBoundCheckerV2.cpp:117
+namespace {
+SVal getDynamicSizeWithOffset(ProgramStateRef State, const MemRegion *MRegion) {
+  SValBuilder &SvalBuilder = State->getStateManager().getSValBuilder();
----------------
Do we need an overload perhaps in `DynamicSize.h` of 
```
SVal getDynamicSizeWithOffset(ProgramStateRef State, const SVal &BufV)
``` 
that takes a MemRegion?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99714



More information about the cfe-commits mailing list