[clang] [clang] Assert the enum FPOpts and LangOpts fit into the storage (PR #126166)

via cfe-commits cfe-commits at lists.llvm.org
Thu Feb 6 17:47:41 PST 2025


github-actions[bot] wrote:

<!--LLVM CODE FORMAT COMMENT: {clang-format}-->


:warning: C/C++ code formatter, clang-format found issues in your code. :warning:

<details>
<summary>
You can test this locally with the following command:
</summary>

``````````bash
git-clang-format --diff 163ccfac33201948eb430db6fac38d265958cfaa 85540ede2e0ba1b6f0b8a69ac6d5a6ddf56fdfc8 --extensions h -- clang/include/clang/Basic/LangOptions.h
``````````

</details>

<details>
<summary>
View the diff from clang-format here.
</summary>

``````````diff
diff --git a/clang/include/clang/Basic/LangOptions.h b/clang/include/clang/Basic/LangOptions.h
index 12621e6bf6..95c1555179 100644
--- a/clang/include/clang/Basic/LangOptions.h
+++ b/clang/include/clang/Basic/LangOptions.h
@@ -648,11 +648,11 @@ public:
 
   // Define accessors/mutators for language options of enumeration type.
 #define LANGOPT(Name, Bits, Default, Description)
-#define ENUM_LANGOPT(Name, Type, Bits, Default, Description) \
-  Type get##Name() const { return static_cast<Type>(Name); } \
-  void set##Name(Type Value) {                               \
-    assert(static_cast<unsigned>(Value) < (1u << Bits));     \
-    Name = static_cast<unsigned>(Value);                     \
+#define ENUM_LANGOPT(Name, Type, Bits, Default, Description)                   \
+  Type get##Name() const { return static_cast<Type>(Name); }                   \
+  void set##Name(Type Value) {                                                 \
+    assert(static_cast<unsigned>(Value) < (1u << Bits));                       \
+    Name = static_cast<unsigned>(Value);                                       \
   }
 #include "clang/Basic/LangOptions.def"
 
@@ -967,7 +967,7 @@ public:
     return static_cast<TYPE>((Value & NAME##Mask) >> NAME##Shift);             \
   }                                                                            \
   void set##NAME(TYPE value) {                                                 \
-    assert(storage_type(value) < (1u << WIDTH));                                             \
+    assert(storage_type(value) < (1u << WIDTH));                               \
     Value = (Value & ~NAME##Mask) | (storage_type(value) << NAME##Shift);      \
   }
 #include "clang/Basic/FPOptions.def"

``````````

</details>


https://github.com/llvm/llvm-project/pull/126166


More information about the cfe-commits mailing list