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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
