[PATCH] D99658: [analyzer] Fix clang_analyzer_getExtent for heap regions
Gabor Marton via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Mar 31 07:17:40 PDT 2021
martong added inline comments.
================
Comment at: clang/lib/StaticAnalyzer/Checkers/ExprInspectionChecker.cpp:258
+ DefinedOrUnknownSVal Size =
+ getDynamicSize(State, MR->getBaseRegion(), C.getSValBuilder());
----------------
martong wrote:
> Wait a bit, I think it should consider the offset too. So, `getDynamicSizeWithOffset` should be used IMHO.
Or we should have a `clang_analyzer_getExtentOfBaseRegion` to be precise.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D99658/new/
https://reviews.llvm.org/D99658
More information about the cfe-commits
mailing list