[clang] 4f96ce4 - [clang][NFC] Convert `Sema::PragmaClangSectionAction` to scoped enum

[Vlad Serebrennikov via cfe-commits]

Author: Vlad Serebrennikov
Date: 2025-04-28T09:06:32+03:00
New Revision: 4f96ce414bcc35ec7e3d8bec18f6a4d7a4c4632a

URL: https://github.com/llvm/llvm-project/commit/4f96ce414bcc35ec7e3d8bec18f6a4d7a4c4632a
DIFF: https://github.com/llvm/llvm-project/commit/4f96ce414bcc35ec7e3d8bec18f6a4d7a4c4632a.diff

LOG: [clang][NFC] Convert `Sema::PragmaClangSectionAction` to scoped enum

Added: 
    

Modified: 
    clang/include/clang/Sema/Sema.h
    clang/lib/Parse/ParsePragma.cpp
    clang/lib/Sema/SemaAttr.cpp

Removed: 
    


################################################################################
diff  --git a/clang/include/clang/Sema/Sema.h b/clang/include/clang/Sema/Sema.h
index 11b6f5934be4f..0b57d50e2b22a 100644
--- a/clang/include/clang/Sema/Sema.h
+++ b/clang/include/clang/Sema/Sema.h
@@ -473,6 +473,8 @@ enum class PragmaClangSectionKind {
   Relro = 5
 };
 
+enum class PragmaClangSectionAction { Set = 0, Clear = 1 };
+
 /// Sema - This implements semantic analysis and AST building for C.
 /// \nosubgrouping
 class Sema final : public SemaBase {
@@ -1422,8 +1424,6 @@ class Sema final : public SemaBase {
   /// Source location for newly created implicit MSInheritanceAttrs
   SourceLocation ImplicitMSInheritanceAttrLoc;
 
-  enum PragmaClangSectionAction { PCSA_Set = 0, PCSA_Clear = 1 };
-
   struct PragmaClangSection {
     std::string SectionName;
     bool Valid = false;

diff  --git a/clang/lib/Parse/ParsePragma.cpp b/clang/lib/Parse/ParsePragma.cpp
index e08364d20a4ab..1d419106bf92b 100644
--- a/clang/lib/Parse/ParsePragma.cpp
+++ b/clang/lib/Parse/ParsePragma.cpp
@@ -2350,11 +2350,11 @@ void PragmaClangSectionHandler::HandlePragma(Preprocessor &PP,
     if (!PP.LexStringLiteral(Tok, SecName, "pragma clang section", false))
       return;
 
-    Actions.ActOnPragmaClangSection(
-        PragmaLocation,
-        (SecName.size() ? Sema::PragmaClangSectionAction::PCSA_Set
-                        : Sema::PragmaClangSectionAction::PCSA_Clear),
-        SecKind, SecName);
+    Actions.ActOnPragmaClangSection(PragmaLocation,
+                                    (SecName.size()
+                                         ? PragmaClangSectionAction::Set
+                                         : PragmaClangSectionAction::Clear),
+                                    SecKind, SecName);
   }
 }
 

diff  --git a/clang/lib/Sema/SemaAttr.cpp b/clang/lib/Sema/SemaAttr.cpp
index cdc71658e259e..bade3a0502a2c 100644
--- a/clang/lib/Sema/SemaAttr.cpp
+++ b/clang/lib/Sema/SemaAttr.cpp
@@ -415,7 +415,7 @@ void Sema::ActOnPragmaClangSection(SourceLocation PragmaLoc,
       llvm_unreachable("invalid clang section kind");
   }
 
-  if (Action == PragmaClangSectionAction::PCSA_Clear) {
+  if (Action == PragmaClangSectionAction::Clear) {
     CSec->Valid = false;
     return;
   }