[PATCH] D54473: [sanitizers] Initial implementation for -fsanitize=init-locals

[Alexander Potapenko via Phabricator via cfe-commits]

glider added a comment.

In https://reviews.llvm.org/D54473#1297460, @kcc wrote:

> This new flag inhibits the warnings from -Wuninitialized, right? 
>  While this is fine for experimenting (and I want to have this in ToT to enable wide experimentation)
>  we should clearly state (in the comments) that the final intent is to make the feature work together with -Wuninitialized.


No, as far as I can see, the warnings from -Wuninitialized are still present (see the test).

> Another thing that we will need (and not necessary in the first change) is to have fine grained controls over what we zero-initialize. 
>  We may want to make separate decisions for pointer/non-pointer scalars, PODs, arrays of {scalars, pointers, PODs}, etc.

Ok, noted.


Repository:
  rC Clang

https://reviews.llvm.org/D54473