[PATCH] D71612: [analyzer] Add PlacementNewChecker

Thu Jan 9 06:46:39 PST 2020

martong added a comment.

In D71612#1790045 <https://reviews.llvm.org/D71612#1790045>, @NoQ wrote:

> I wonder if this checker will find any misuses of placement into `llvm::TrailingObjects`.

I've evaluated this checker on LLVM/Clang source and it did not find any results, though there are many placement new calls there. I think this is good, because there are no false positives.

  [INFO 2019-12-21 00:15] - ----==== Summary ====----
  [INFO 2019-12-21 00:15] - Successfully analyzed
  [INFO 2019-12-21 00:15] -   clangsa: 2975
  [INFO 2019-12-21 00:15] - Failed to analyze
  [INFO 2019-12-21 00:15] -   clangsa: 2

The two failures are caused by non existing `.inc` files, e.g.:

  /mnt/ssd/egbomrt/WORK/llvm1/git/llvm-project/llvm/unittests/Option/OptionParsingTest.cpp:23:10: fatal error: 'Opts.inc' file not found
  #include "Opts.inc"
           ^~~~~~~~~~
  1 error generated.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71612