Details
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Marking as "request changes" to make sure we don't forget to discuss the "nit" :)
llvm/test/CodeGen/AArch64/sve-intrinsics-perm-select-matmul-fp64.ll | ||
---|---|---|
1 | 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. |
llvm/test/CodeGen/AArch64/sve-intrinsics-perm-select-matmul-fp64.ll | ||
---|---|---|
1 | FWIW, we could keep everything in one test without adding the extra +bf16 at command line invocation and by using a function attribute attached to the bfloat functions, as I have done in https://reviews.llvm.org/D82429 |
llvm/test/CodeGen/AArch64/sve-intrinsics-perm-select-matmul-fp64.ll | ||
---|---|---|
1 |
I like this approach, I'll go with that |
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.