[PATCH] D144977: [analyzer] Fix of the initialization list parsing.

2023-03-02 Thread Domján Dániel via Phabricator via cfe-commits
isuckatcs added inline comments. Comment at: clang/lib/StaticAnalyzer/Checkers/CStringChecker.cpp:432 + const LocationContext *LCtx = C.getLocationContext(); + SVal SrcVal = State->getSVal(Buffer.Expression, LCtx); + QualType SourceValType = SrcVal.getType(C.getAST

[PATCH] D144977: [analyzer] Fix of the initialization list parsing.

2023-03-02 Thread Eänolituri Lómitaurë via Phabricator via cfe-commits
earnol requested review of this revision. earnol added inline comments. Comment at: clang/lib/StaticAnalyzer/Core/RegionStore.cpp:1787 + if (V && + (!targetType->isStructureOrClassType() && !targetType->isUnionType())) return *V; isuckatcs wrote: > I a

[PATCH] D144977: [analyzer] Fix of the initialization list parsing.

2023-03-02 Thread Eänolituri Lómitaurë via Phabricator via cfe-commits
earnol planned changes to this revision. earnol added inline comments. Comment at: clang/lib/StaticAnalyzer/Core/RegionStore.cpp:564-565 - SVal getBindingForElement(RegionBindingsConstRef B, const ElementRegion *R); + SVal getBindingForElement(RegionBindingsConstRef B, const

[PATCH] D144977: [analyzer] Fix of the initialization list parsing.

2023-03-02 Thread Domján Dániel via Phabricator via cfe-commits
isuckatcs added inline comments. Comment at: clang/lib/StaticAnalyzer/Core/RegionStore.cpp:1787 + if (V && + (!targetType->isStructureOrClassType() && !targetType->isUnionType())) return *V; I assume `targetType` is the type we want to interpret the re

[PATCH] D144977: [analyzer] Fix of the initialization list parsing.

2023-03-01 Thread Artem Dergachev via Phabricator via cfe-commits
NoQ added a comment. Hi, looks very interesting! We definitely have troubles with some initializer lists. It looks like you're primarily interested in the alpha/unsupported checker, but your patch probably affects behavior of the default setup as well. If you've found any difference in results

[PATCH] D144977: [analyzer] Fix of the initialization list parsing.

2023-02-28 Thread Eänolituri Lómitaurë via Phabricator via cfe-commits
earnol created this revision. Herald added subscribers: steakhal, manas, ASDenysPetrov, martong, dkrupp, donat.nagy, Szelethus, mikhail.ramalho, a.sidorin, szepet, baloghadamsoftware, xazax.hun. Herald added a reviewer: NoQ. Herald added a project: All. earnol requested review of this revision. H