[clang] [llvm] [SystemZ] Add support for half (fp16) (PR #109164)

via cfe-commits cfe-commits at lists.llvm.org
Wed Sep 18 08:48:52 PDT 2024


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 8411214c56eb3f9fb77fe3a9f156d4e6ef6cec06 e0bb2adac31f9bec6fa1521a5f9fd67ce7442e14 --extensions h,cpp -- clang/lib/Basic/Targets/SystemZ.h llvm/lib/Target/SystemZ/SystemZISelLowering.cpp
``````````

</details>

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

``````````diff
diff --git a/clang/lib/Basic/Targets/SystemZ.h b/clang/lib/Basic/Targets/SystemZ.h
index 6566b63d45..e61ceaf8c0 100644
--- a/clang/lib/Basic/Targets/SystemZ.h
+++ b/clang/lib/Basic/Targets/SystemZ.h
@@ -92,18 +92,16 @@ public:
     }
     MaxAtomicPromoteWidth = MaxAtomicInlineWidth = 128;
 
-    HasLegalHalfType = false;    // Default=false
-    HalfArgsAndReturns = false;  // Default=false
-    HasFloat16 = true;           // Default=false
+    HasLegalHalfType = false;   // Default=false
+    HalfArgsAndReturns = false; // Default=false
+    HasFloat16 = true;          // Default=false
 
     HasStrictFP = true;
   }
 
   unsigned getMinGlobalAlign(uint64_t Size, bool HasNonWeakDef) const override;
 
-  bool useFP16ConversionIntrinsics() const override {
-    return false;
-  }
+  bool useFP16ConversionIntrinsics() const override { return false; }
 
   void getTargetDefines(const LangOptions &Opts,
                         MacroBuilder &Builder) const override;

``````````

</details>


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


More information about the cfe-commits mailing list