[PATCH] D151214: [clang][Sema] `-Wshadow` warns about shadowings by static local variables

2023-05-24 Thread Takuya Shimizu via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG29dc47a9eeeb: [clang][Sema] `-Wshadow` warns about shadowings by static local variables (authored by hazohelet). Changed prior to commit: https://reviews.llvm.org/D151214?vs=524709=525126#toc

[PATCH] D151214: [clang][Sema] `-Wshadow` warns about shadowings by static local variables

2023-05-23 Thread Shafik Yaghmour via Phabricator via cfe-commits
shafik accepted this revision. shafik added a comment. This revision is now accepted and ready to land. LGTM Comment at: clang/test/SemaCXX/warn-shadow.cpp:33 void foo() { int i; // expected-warning {{declaration shadows a variable in namespace '(anonymous)'}} int j;

[PATCH] D151214: [clang][Sema] `-Wshadow` warns about shadowings by static local variables

2023-05-23 Thread Timm Bäder via Phabricator via cfe-commits
tbaeder added inline comments. Comment at: clang/docs/ReleaseNotes.rst:295 +- Clang's `-Wshadow` warning now warns about shadowings by static local variables + (`#62850: `_, Repository: rG LLVM Github

[PATCH] D151214: [clang][Sema] `-Wshadow` warns about shadowings by static local variables

2023-05-23 Thread Takuya Shimizu via Phabricator via cfe-commits
hazohelet created this revision. hazohelet added reviewers: aaron.ballman, tbaeder, shafik. Herald added a project: All. hazohelet requested review of this revision. Herald added a project: clang. This patch makes `-Wshadow` warn about the shadowings by static local variables. Fixes