stephanemoore added a comment.

> Does the check allow variables in anonymous namespaces?
> 
>   namespace {
>   dispatch_once_t onceToken;
>   }
> 
> 
> I //think// such variables should satisfy initialization requirements.
> 
> If not, can we update the check to not trigger on variables? Either way, can 
> we add a test to verify behavior for variables in anonymous namespaces?

Nevermind, I overlooked that there is already a variable in an anonymous 
namespace in the test 😅 Thanks for that!


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

https://reviews.llvm.org/D67567



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to