[PATCH] D129544: [RISCV][test] Add test of binop followed by extractelement.

[Jianjian Guan via Phabricator via llvm-commits]

jacquesguan marked 3 inline comments as done.
jacquesguan added inline comments.


================
Comment at: llvm/test/CodeGen/RISCV/rvv/extractelt-fp-rv32.ll:485
+
+define float @ext_fadd_nxv4f32(<vscale x 4 x float> %x) {
+; CHECK-LABEL: ext_fadd_nxv4f32:
----------------
reames wrote:
> Naming wise, "ext" makes me think "extend" not "extractelement".
> 
> I'd suggest extractelt_<type>_<op>_splat.  
Done.


================
Comment at: llvm/test/CodeGen/RISCV/rvv/fixed-vectors-extract.ll:653
+; CHECK-NEXT:    ret
+  %bo = mul <4 x i32> %x, <i32 1, i32 2, i32 3, i32 42>
+  %ext = extractelement <4 x i32> %bo, i32 3
----------------
reames wrote:
> Meta comment: "1" and "0" are special edge cases for mul and div.  Please use a constant which is not an edge case if you're not testing that behavior.
Done.


================
Comment at: llvm/test/CodeGen/RISCV/rvv/fixed-vectors-extract.ll:662
+; CHECK-NEXT:    vsetivli zero, 0, e32, m1, ta, mu
+; CHECK-NEXT:    vmv.x.s a0, v8
+; CHECK-NEXT:    ret
----------------
reames wrote:
> Example of previous comment causing less than helpful test output.  
Done.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D129544/new/

https://reviews.llvm.org/D129544