[llvm] d0278cf - [RISCV] Remove some more completed FIXMEs from tests. NFC
Luke Lau via llvm-commits
llvm-commits at lists.llvm.org
Thu Sep 5 03:13:04 PDT 2024
Author: Luke Lau
Date: 2024-09-05T18:12:49+08:00
New Revision: d0278cf395e09bfb8dbef9cb92e6103be91e1eb3
URL: https://github.com/llvm/llvm-project/commit/d0278cf395e09bfb8dbef9cb92e6103be91e1eb3
DIFF: https://github.com/llvm/llvm-project/commit/d0278cf395e09bfb8dbef9cb92e6103be91e1eb3.diff
LOG: [RISCV] Remove some more completed FIXMEs from tests. NFC
Added:
Modified:
llvm/test/CodeGen/RISCV/rvv/fixed-vectors-insert.ll
llvm/test/CodeGen/RISCV/rvv/vadd-vp.ll
llvm/test/CodeGen/RISCV/rvv/vmax-vp.ll
llvm/test/CodeGen/RISCV/rvv/vmaxu-vp.ll
llvm/test/CodeGen/RISCV/rvv/vmin-vp.ll
llvm/test/CodeGen/RISCV/rvv/vminu-vp.ll
llvm/test/CodeGen/RISCV/rvv/vwadd-sdnode.ll
Removed:
################################################################################
diff --git a/llvm/test/CodeGen/RISCV/rvv/fixed-vectors-insert.ll b/llvm/test/CodeGen/RISCV/rvv/fixed-vectors-insert.ll
index 776a1e9bab6b26..81fb86cd81cd35 100644
--- a/llvm/test/CodeGen/RISCV/rvv/fixed-vectors-insert.ll
+++ b/llvm/test/CodeGen/RISCV/rvv/fixed-vectors-insert.ll
@@ -47,7 +47,6 @@ define <32 x i32> @insertelt_v32i32_0(<32 x i32> %a, i32 %y) {
ret <32 x i32> %b
}
-; FIXME: Should only require an m2 slideup
define <32 x i32> @insertelt_v32i32_4(<32 x i32> %a, i32 %y) {
; CHECK-LABEL: insertelt_v32i32_4:
; CHECK: # %bb.0:
diff --git a/llvm/test/CodeGen/RISCV/rvv/vadd-vp.ll b/llvm/test/CodeGen/RISCV/rvv/vadd-vp.ll
index 2a4fbb248cd9c1..feeef73e538ae0 100644
--- a/llvm/test/CodeGen/RISCV/rvv/vadd-vp.ll
+++ b/llvm/test/CodeGen/RISCV/rvv/vadd-vp.ll
@@ -1425,7 +1425,6 @@ define <vscale x 32 x i32> @vadd_vi_nxv32i32_evl_nx8(<vscale x 32 x i32> %va, <v
ret <vscale x 32 x i32> %v
}
-; FIXME: The first vadd.vi should be able to infer that its AVL is equivalent to VLMAX.
; FIXME: The upper half of the operation is doing nothing but we don't catch
; that on RV64; we issue a usubsat(and (vscale x 16), 0xffffffff, vscale x 16)
; (the "original" %evl is the "and", due to known-bits issues with legalizing
diff --git a/llvm/test/CodeGen/RISCV/rvv/vmax-vp.ll b/llvm/test/CodeGen/RISCV/rvv/vmax-vp.ll
index 5fdfb332da7cf8..f65e708f5303cc 100644
--- a/llvm/test/CodeGen/RISCV/rvv/vmax-vp.ll
+++ b/llvm/test/CodeGen/RISCV/rvv/vmax-vp.ll
@@ -1062,7 +1062,6 @@ define <vscale x 32 x i32> @vmax_vx_nxv32i32_evl_nx8(<vscale x 32 x i32> %va, i3
ret <vscale x 32 x i32> %v
}
-; FIXME: The first vmax.vx should be able to infer that its AVL is equivalent to VLMAX.
; FIXME: The upper half of the operation is doing nothing but we don't catch
; that on RV64; we issue a usubsat(and (vscale x 16), 0xffffffff, vscale x 16)
; (the "original" %evl is the "and", due to known-bits issues with legalizing
diff --git a/llvm/test/CodeGen/RISCV/rvv/vmaxu-vp.ll b/llvm/test/CodeGen/RISCV/rvv/vmaxu-vp.ll
index 7d678950b7a3c1..df1ad58e5ecbde 100644
--- a/llvm/test/CodeGen/RISCV/rvv/vmaxu-vp.ll
+++ b/llvm/test/CodeGen/RISCV/rvv/vmaxu-vp.ll
@@ -1061,7 +1061,6 @@ define <vscale x 32 x i32> @vmaxu_vx_nxv32i32_evl_nx8(<vscale x 32 x i32> %va, i
ret <vscale x 32 x i32> %v
}
-; FIXME: The first vmaxu.vx should be able to infer that its AVL is equivalent to VLMAX.
; FIXME: The upper half of the operation is doing nothing but we don't catch
; that on RV64; we issue a usubsat(and (vscale x 16), 0xffffffff, vscale x 16)
; (the "original" %evl is the "and", due to known-bits issues with legalizing
diff --git a/llvm/test/CodeGen/RISCV/rvv/vmin-vp.ll b/llvm/test/CodeGen/RISCV/rvv/vmin-vp.ll
index 98a288ed68b9ac..0bf0638633aa45 100644
--- a/llvm/test/CodeGen/RISCV/rvv/vmin-vp.ll
+++ b/llvm/test/CodeGen/RISCV/rvv/vmin-vp.ll
@@ -1062,7 +1062,6 @@ define <vscale x 32 x i32> @vmin_vx_nxv32i32_evl_nx8(<vscale x 32 x i32> %va, i3
ret <vscale x 32 x i32> %v
}
-; FIXME: The first vmin.vx should be able to infer that its AVL is equivalent to VLMAX.
; FIXME: The upper half of the operation is doing nothing but we don't catch
; that on RV64; we issue a usubsat(and (vscale x 16), 0xffffffff, vscale x 16)
; (the "original" %evl is the "and", due to known-bits issues with legalizing
diff --git a/llvm/test/CodeGen/RISCV/rvv/vminu-vp.ll b/llvm/test/CodeGen/RISCV/rvv/vminu-vp.ll
index 34b554b7ff5145..2acebdf2e646d4 100644
--- a/llvm/test/CodeGen/RISCV/rvv/vminu-vp.ll
+++ b/llvm/test/CodeGen/RISCV/rvv/vminu-vp.ll
@@ -1061,7 +1061,6 @@ define <vscale x 32 x i32> @vminu_vx_nxv32i32_evl_nx8(<vscale x 32 x i32> %va, i
ret <vscale x 32 x i32> %v
}
-; FIXME: The first vminu.vx should be able to infer that its AVL is equivalent to VLMAX.
; FIXME: The upper half of the operation is doing nothing but we don't catch
; that on RV64; we issue a usubsat(and (vscale x 16), 0xffffffff, vscale x 16)
; (the "original" %evl is the "and", due to known-bits issues with legalizing
diff --git a/llvm/test/CodeGen/RISCV/rvv/vwadd-sdnode.ll b/llvm/test/CodeGen/RISCV/rvv/vwadd-sdnode.ll
index a4d58985b75de5..b7ce0e3f196f05 100644
--- a/llvm/test/CodeGen/RISCV/rvv/vwadd-sdnode.ll
+++ b/llvm/test/CodeGen/RISCV/rvv/vwadd-sdnode.ll
@@ -1394,8 +1394,6 @@ define <vscale x 1 x i64> @i1_zext(<vscale x 1 x i1> %va, <vscale x 1 x i64> %vb
}
; %x.i32 and %y.i32 are disjoint, so DAGCombiner will combine it into an or.
-; FIXME: We should be able to recover the or into vwaddu.vv if the disjoint
-; flag is set.
define <vscale x 2 x i32> @vwaddu_vv_disjoint_or_add(<vscale x 2 x i8> %x.i8, <vscale x 2 x i8> %y.i8) {
; CHECK-LABEL: vwaddu_vv_disjoint_or_add:
; CHECK: # %bb.0:
More information about the llvm-commits
mailing list