[llvm] [TableGen] Avoid deprecated-enum-enum-conversion C++20 warnings; NFC (PR #218909)

Sven van Haastregt via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 22 04:03:19 PDT 2026


https://github.com/svenvh updated https://github.com/llvm/llvm-project/pull/218909

>From d2296d05a50c1f9b939cb886cf7e67f2bd3187d9 Mon Sep 17 00:00:00 2001
From: Sven van Haastregt <sven.vanhaastregt at arm.com>
Date: Tue, 25 Aug 2026 14:59:40 +0100
Subject: [PATCH] [TableGen] Avoid deprecated-enum-enum-conversion C++20
 warnings; NFC
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

The bitwise OR of different enum types resulted in many C++20
deprecation warnings in the generated option tables:

     warning: bitwise operation between different enumeration types
     ‘llvm::opt::DriverVisibility’ and
     ‘clang::options::ClangVisibility’ is deprecated
     [-Wdeprecated-enum-enum-conversion]

Fix by emitting casts to `unsigned`, matching the types of the
corresponding fields in `OptTable::Info`.
---
 llvm/utils/TableGen/OptionParserEmitter.cpp | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/llvm/utils/TableGen/OptionParserEmitter.cpp b/llvm/utils/TableGen/OptionParserEmitter.cpp
index dd13c927b2e02..2360d55267625 100644
--- a/llvm/utils/TableGen/OptionParserEmitter.cpp
+++ b/llvm/utils/TableGen/OptionParserEmitter.cpp
@@ -301,10 +301,12 @@ static void emitOptionParser(const RecordKeeper &Records, raw_ostream &OS) {
     raw_string_ostream MaskOS(Mask);
     ListSeparator Sep(" | ");
     for (const Init *I : *R.getValueAsListInit(Field))
-      MaskOS << Sep << cast<DefInit>(I)->getDef()->getName();
+      MaskOS << Sep << "static_cast<unsigned>("
+             << cast<DefInit>(I)->getDef()->getName() << ")";
     if (const DefInit *DI = dyn_cast<DefInit>(R.getValueInit("Group")))
       for (const Init *I : *DI->getDef()->getValueAsListInit(Field))
-        MaskOS << Sep << cast<DefInit>(I)->getDef()->getName();
+        MaskOS << Sep << "static_cast<unsigned>("
+               << cast<DefInit>(I)->getDef()->getName() << ")";
     return Mask.empty() ? std::string("0") : Mask;
   };
 



More information about the llvm-commits mailing list