[PATCH] D66364: Diagnose use of _Thread_local as an extension when appropriate
Richard Smith - zygoloid via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Aug 16 15:31:57 PDT 2019
rsmith added a comment.
`_Thread_local` is a reserved identifier; we generally don't produce extension warnings for uses of reserved identifiers. (Eg, there's no warning for `_Atomic` in C++ or `_Bool` in C89, and no warning for uses of `__type_traits` or `__builtins`.)
But I note that we *do* warn for some of these already (eg, `_Generic`, `_Static_assert`, `_Alignas`, and `_Alignof` get a warning). We should choose a rule and apply it consistently.
What's the motivation for warning on this? Maybe that can inform whether these warnings are useful in general.
CHANGES SINCE LAST ACTION
More information about the cfe-commits