[PATCH] D82492: [AArch64][SVE] Guard perm and select bfloat16 intrinsic patterns
Francesco Petrogalli via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Jun 24 13:01:47 PDT 2020
fpetrogalli requested changes to this revision.
fpetrogalli added a comment.
This revision now requires changes to proceed.
Marking as "request changes" to make sure we don't forget to discuss the "nit" :)
================
Comment at: llvm/test/CodeGen/AArch64/sve-intrinsics-perm-select-matmul-fp64.ll:1
-; RUN: llc -mtriple=aarch64-none-linux-gnu -mattr=+sve,+f64mm -asm-verbose=0 < %s -o - | FileCheck %s
+; RUN: llc -mtriple=aarch64-none-linux-gnu -mattr=+sve,+f64mm,+bf16 -asm-verbose=0 < %s -o - | FileCheck %s
----------------
Nit: given that the fact that we have added the bfloat tests in the same file as the non-bfloat tests might have contributes to overseeing the problem, I wonder if it make sense to use the same approach we have done for the ACLE tests and have the tests requiring `+bf16` to be extracted into a separate `*-bfloat` file.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D82492/new/
https://reviews.llvm.org/D82492
More information about the llvm-commits
mailing list