[PATCH] D71001: [Clang-Tidy] New check: bugprone-misplaced-pointer-arithmetic-in-alloc

Dmitri Gribenko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Dec 4 10:50:05 PST 2019


gribozavr2 added a comment.

In D71001#1769071 <https://reviews.llvm.org/D71001#1769071>, @baloghadamsoftware wrote:

> In D71001#1769018 <https://reviews.llvm.org/D71001#1769018>, @gribozavr2 wrote:
>
> > ASan can help debug this issue, and more.
>
>
> ASan is too heavyweight for this simple problem. It does not point out the source of the issue as quickly as this simple check which also provides a fix. ASan is meant for the less trivial cases. Is this really such a performance hit? Clang-Tidy already contains lots of checks which target a very narrow pattern.


It is not just a performance hit. Adding a new checker is primarily a maintenance burden.


Repository:
  rCTE Clang Tools Extra

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

https://reviews.llvm.org/D71001





More information about the cfe-commits mailing list