[PATCH] D76996: [analyzer] Improve PlacementNewChecker

Karasev Nikita via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Mar 30 10:17:31 PDT 2020


f00kat updated this revision to Diff 253628.
f00kat marked 3 inline comments as done.
f00kat added a comment.

1. Removed code and tests for ConcreteInt cases
2. Fixed FieldRegion check
3. Added handling for ElementRegion cases such as

  void f7() {
    short b[10];
  
    // ok. 2(short align) + 3*2(index '1' offset)
    ::new (&b[3]) long;
  }

4. Fixed align error message
5. Maybe fixed lint warnings


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76996

Files:
  clang/lib/StaticAnalyzer/Checkers/CheckPlacementNew.cpp
  clang/test/Analysis/placement-new.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D76996.253628.patch
Type: text/x-patch
Size: 14992 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20200330/3b3d1e06/attachment-0001.bin>


More information about the cfe-commits mailing list