[PATCH] D123579: [RISCV][VP] Add RVV codegen for vp.trunc.

[Fraser Cormack via Phabricator via llvm-commits]

frasercrmck added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVISelLowering.cpp:3172
   }
   case ISD::TRUNCATE: {
     // Only custom-lower vector truncates
----------------
Can trim the unnecessary `{}` now too


================
Comment at: llvm/test/CodeGen/RISCV/rvv/fixed-vector-trunc-vp-mask.ll:2
+; NOTE: Assertions have been autogenerated by utils/update_llc_test_checks.py
+; RUN: llc -mtriple=riscv32 -mattr=+v -riscv-v-vector-bits-min=128 < %s | FileCheck %s
+; RUN: llc -mtriple=riscv64 -mattr=+v -riscv-v-vector-bits-min=128 < %s | FileCheck %s
----------------
`-verify-machineinstrs` wouldn't hurt


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D123579