[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