[PATCH] D70265: [clang-tidy] Add CppCoreGuidelines I.2 "Avoid non-const global variables" check

[Roman Lebedev via Phabricator via cfe-commits]

lebedev.ri added a comment.

In D70265#1954925 <https://reviews.llvm.org/D70265#1954925>, @vingeldal wrote:

> After looking more closely at the code I think the issue is within hasLocalStorage() which is called in hasGlobalStorage(). My expectation would be that anything inside of function scope would be considered local but I'm not very certain.
>  Any thoughts on whether hasLocalStorage() should be modified or if I should change the check and use some more ad-hoc implementation, instead of hasGlobalStorage(), to determine if the variable is local or global?


It's not ideal to have known-false-positive for long, so please post //some// fix, i think we'll figure it out there.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70265