[clang] clang: Add pragma clang fp reciprocal (PR #68267)

via cfe-commits cfe-commits at lists.llvm.org
Fri Nov 3 01:23:06 PDT 2023


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 8e2b3309a975748649a504620a9600da9fe3c837 db9b84992dbd6d75dc5c23b11d63c195400d5bc1 -- clang/test/CodeGen/fp-reciprocal-pragma.cpp clang/include/clang/Basic/PragmaKinds.h clang/include/clang/Sema/Sema.h clang/lib/Parse/ParsePragma.cpp clang/lib/Sema/SemaAttr.cpp clang/test/Parser/pragma-fp-contract.c clang/test/Parser/pragma-fp.cpp clang/test/Sema/eval-method-with-unsafe-math.c
``````````

</details>

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

``````````diff
diff --git a/clang/include/clang/Basic/PragmaKinds.h b/clang/include/clang/Basic/PragmaKinds.h
index a4789ccde5ec..42f049f7323d 100644
--- a/clang/include/clang/Basic/PragmaKinds.h
+++ b/clang/include/clang/Basic/PragmaKinds.h
@@ -36,13 +36,12 @@ enum PragmaFloatControlKind {
 };
 
 enum PragmaFPKind {
-  PFK_Contract, // #pragma clang fp contract
+  PFK_Contract,    // #pragma clang fp contract
   PFK_Reassociate, // #pragma clang fp reassociate
   PFK_Reciprocal,  // #pragma clang fp reciprocal
-  PFK_Exceptions, // #pragma clang fp exceptions
-  PFK_EvalMethod // #pragma clang fp eval_method
+  PFK_Exceptions,  // #pragma clang fp exceptions
+  PFK_EvalMethod   // #pragma clang fp eval_method
 };
-
 }
 
 #endif
diff --git a/clang/lib/Parse/ParsePragma.cpp b/clang/lib/Parse/ParsePragma.cpp
index f0d366aaa134..9a515bf333aa 100644
--- a/clang/lib/Parse/ParsePragma.cpp
+++ b/clang/lib/Parse/ParsePragma.cpp
@@ -3219,8 +3219,7 @@ void PragmaFPHandler::HandlePragma(Preprocessor &PP,
     IdentifierInfo *OptionInfo = Tok.getIdentifierInfo();
 
     auto FlagKind =
-        llvm::StringSwitch<std::optional<PragmaFPKind>>(
-            OptionInfo->getName())
+        llvm::StringSwitch<std::optional<PragmaFPKind>>(OptionInfo->getName())
             .Case("contract", PFK_Contract)
             .Case("reassociate", PFK_Reassociate)
             .Case("exceptions", PFK_Exceptions)
@@ -3265,11 +3264,9 @@ void PragmaFPHandler::HandlePragma(Preprocessor &PP,
             << PP.getSpelling(Tok) << OptionInfo->getName() << *FlagKind;
         return;
       }
-    } else if (FlagKind == PFK_Reassociate ||
-               FlagKind == PFK_Reciprocal) {
-      auto &Value = FlagKind == PFK_Reassociate
-                        ? AnnotValue->ReassociateValue
-                        : AnnotValue->ReciprocalValue;
+    } else if (FlagKind == PFK_Reassociate || FlagKind == PFK_Reciprocal) {
+      auto &Value = FlagKind == PFK_Reassociate ? AnnotValue->ReassociateValue
+                                                : AnnotValue->ReciprocalValue;
       Value = llvm::StringSwitch<std::optional<LangOptions::FPModeKind>>(
                   II->getName())
                   .Case("on", LangOptions::FPModeKind::FPM_On)

``````````

</details>


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


More information about the cfe-commits mailing list