[PATCH] D109377: type legalization for vp_load and vp_store.
Fraser Cormack via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Sep 10 02:02:35 PDT 2021
frasercrmck added inline comments.
================
Comment at: llvm/test/CodeGen/Generic/fixed-vector-vp-mem-legalization.ll:2
+; NOTE: Assertions have been autogenerated by utils/update_llc_test_checks.py
+; RUN: llc -mtriple=riscv64 -mattr=+experimental-v -riscv-v-vector-bits-min=128 -verify-machineinstrs < %s | FileCheck %s
+
----------------
We shouldn't put RISC-V tests in `Generic` unless we know the target is built. So either this needs to use `REQUIRES:` or we put it in `CodeGen/RISCV`. I prefer the latter as that's where I've seen all other target-specific legalization tests go.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D109377/new/
https://reviews.llvm.org/D109377
More information about the llvm-commits
mailing list