[llvm-dev] [cfe-dev] Warnings after running PREfast on LLVM
David Blaikie via llvm-dev
llvm-dev at lists.llvm.org
Wed Apr 22 15:45:40 PDT 2020
I'd guess that there's not much interest in fixing shadowing - that not
enough of them are actually buggy (represent cases where the developer
intended something other than what they wrote/what the compiler understood).
On Wed, Apr 22, 2020 at 3:36 PM Mandeep Singh Grang via cfe-dev <
cfe-dev at lists.llvm.org> wrote:
> As part of compliance for our Checked C
> <https://github.com/microsoft/checkedc-clang> LLVM compiler, we at
> Microsoft ran the PREfast
> <https://www.codeproject.com/Articles/167588/Using-PREfast-for-Static-Code-Analysis> static
> analysis tool on the entire llvm/clang codebase. The tool reported hundreds
> of warnings due to variables being shadowed.For example:
> clang/lib/Sema/TreeTransform.h(3709) Local declaration of 'I' hides
> declaration of the same name in outer scope. See previous declaration at
> line '3653'.
> We can upstream fixes to these (and other) warnings if the community would
> like them fixed. The full list of warnings can be found here
> Also, is there a reason we do not turn on the -Wshadow flag by default for
> llvm builds?
> cfe-dev mailing list
> cfe-dev at lists.llvm.org
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the llvm-dev