[PATCH] D76229: [clang-tidy] Added PlacementNewStorageCheck

Roman Lebedev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Mar 16 14:46:31 PDT 2020


lebedev.ri requested changes to this revision.
lebedev.ri added a comment.
This revision now requires changes to proceed.

This seems to be already handled by clang static analyzer? (`clang-analyzer-cplusplus.PlacementNew`)

  <source>:19:5: warning: Storage provided to placement new is only 2 bytes, whereas the allocated type requires 8 bytes [clang-analyzer-cplusplus.PlacementNew]
      ::new (&s1) long;
      ^
  <source>:19:5: note: Storage provided to placement new is only 2 bytes, whereas the allocated type requires 8 bytes
  <source>:64:3: warning: Storage provided to placement new is only 2 bytes, whereas the allocated type requires 8 bytes [clang-analyzer-cplusplus.PlacementNew]
    ::new (buffer3) long;
    ^

https://godbolt.org/z/9VX5WW


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76229





More information about the cfe-commits mailing list