[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