================
@@ -236,6 +236,7 @@ FEATURE(shadow_call_stack,
 FEATURE(tls, PP.getTargetInfo().isTLSSupported())
 FEATURE(underlying_type, LangOpts.CPlusPlus)
 FEATURE(experimental_library, LangOpts.ExperimentalLibrary)
+FEATURE(datasizeof, LangOpts.CPlusPlus)
----------------
AaronBallman wrote:

Yeahhhhh, but we still need to diagnose it as an extension because that's what 
it is. (We should not add extensions that we do not diagnose under `-pedantic` 
-- we have been inconsistent in the past but we've gotten better in the past 
several years.)

Pedantic warnings aren't triggered in system headers, so I think I'm failing to 
see the concern: https://godbolt.org/z/nn7TdKrcv

https://github.com/llvm/llvm-project/pull/67805
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to