[clang] [clang][Diagnostic] Don't warn about binary literals when using C23. (PR #80244)

Collin Funk via cfe-commits cfe-commits at lists.llvm.org
Wed Jan 31 21:34:47 PST 2024


https://github.com/collinfunk created https://github.com/llvm/llvm-project/pull/80244

The C23 standard brought binary literals which were previously GNU extensions. Silence warnings from -Wgnu-binary-literal when using C23. This warning is implied by -Wpedantic.

>From ef443507e06adcbc604ad8e03d96060e29525394 Mon Sep 17 00:00:00 2001
From: Collin Funk <collin.funk1 at gmail.com>
Date: Wed, 31 Jan 2024 21:30:39 -0800
Subject: [PATCH] [clang][Diagnostic] Don't warn about binary literals when
 using C23.

The C23 standard brought binary literals which were previously GNU
extensions. Silence warnings from -Wgnu-binary-literal when using C23.
This warning is implied by -Wpedantic.

Signed-off-by: Collin Funk <collin.funk1 at gmail.com>
---
 clang/lib/Lex/LiteralSupport.cpp | 16 +++++++++++-----
 1 file changed, 11 insertions(+), 5 deletions(-)

diff --git a/clang/lib/Lex/LiteralSupport.cpp b/clang/lib/Lex/LiteralSupport.cpp
index 0a78638f68051..add4ca76678de 100644
--- a/clang/lib/Lex/LiteralSupport.cpp
+++ b/clang/lib/Lex/LiteralSupport.cpp
@@ -1358,11 +1358,17 @@ void NumericLiteralParser::ParseNumberStartingWithZero(SourceLocation TokLoc) {
 
   // Handle simple binary numbers 0b01010
   if ((c1 == 'b' || c1 == 'B') && (s[1] == '0' || s[1] == '1')) {
-    // 0b101010 is a C++1y / GCC extension.
-    Diags.Report(TokLoc, LangOpts.CPlusPlus14
-                             ? diag::warn_cxx11_compat_binary_literal
-                         : LangOpts.CPlusPlus ? diag::ext_binary_literal_cxx14
-                                              : diag::ext_binary_literal);
+    // 0b101010 is a GCC extension subsequently standardized by C23 and C++14.
+    if (LangOpts.CPlusPlus) {
+      if (LangOpts.CPlusPlus14)
+        Diags.Report(TokLoc, diag::warn_cxx11_compat_binary_literal);
+      else
+        Diags.Report(TokLoc, diag::ext_binary_literal_cxx14);
+    } else {
+      // TODO: Need new DiagGroup for C23.
+      if (!LangOpts.C23)
+        Diags.Report(TokLoc, diag::ext_binary_literal);
+    }
     ++s;
     assert(s < ThisTokEnd && "didn't maximally munch?");
     radix = 2;



More information about the cfe-commits mailing list