[clang] [RISCV][NFC] Remove unneeded defining name of `vundefined` (PR #101643)

via cfe-commits cfe-commits at lists.llvm.org
Fri Aug 2 02:18:39 PDT 2024


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-clang

Author: Brandon Wu (4vtomat)

<details>
<summary>Changes</summary>

`vundefined` doesn't have corresponding named IR, instead it generates
`poison` value, we already define the `Name` for C intrinsics, so we
don't need the defining name at all.


---
Full diff: https://github.com/llvm/llvm-project/pull/101643.diff


1 Files Affected:

- (modified) clang/include/clang/Basic/riscv_vector.td (+3-3) 


``````````diff
diff --git a/clang/include/clang/Basic/riscv_vector.td b/clang/include/clang/Basic/riscv_vector.td
index a0820e2093bc2..13d2efb4e6109 100644
--- a/clang/include/clang/Basic/riscv_vector.td
+++ b/clang/include/clang/Basic/riscv_vector.td
@@ -2407,10 +2407,10 @@ let HasMasked = false, HasVL = false, IRName = "" in {
       ManualCodegen = [{
         return llvm::PoisonValue::get(ResultType);
       }] in {
-    def vundefined : RVVBuiltin<"v", "v", "csilxfd">;
+    def : RVVBuiltin<"v", "v", "csilxfd">;
     let RequiredFeatures = ["Zvfbfmin"] in
-      def vundefined_bf16 : RVVBuiltin<"v", "v", "y">;
-    def vundefined_u : RVVBuiltin<"Uv", "Uv", "csil">;
+      def : RVVBuiltin<"v", "v", "y">;
+    def : RVVBuiltin<"Uv", "Uv", "csil">;
 
     foreach nf = NFList in {
       let NF = nf in {

``````````

</details>


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


More information about the cfe-commits mailing list