[PATCH] D136858: [AArch64-SVE]: Force generating code compatible to streaming mode for sve-fixed-length tests.
Hassnaa Hamdi via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Oct 27 10:25:39 PDT 2022
hassnaa-arm added inline comments.
================
Comment at: llvm/test/CodeGen/AArch64/sve-streaming-mode-fixed-length-bitselect.ll:11
+;
+
+define <8 x i32> @fixed_bitselect_v8i32(<8 x i32>* %pre_cond_ptr, <8 x i32>* %left_ptr, <8 x i32>* %right_ptr) #0 {
----------------
I left that comment intentionally to choose which solution is better, the current solution (disable combining or into BSP), or implement SVE lowering for the BSP pseudoinst as the comment suggest.
================
Comment at: llvm/test/CodeGen/AArch64/sve-streaming-mode-fixed-length-bitselect.ll:47
+
+attributes #0 = { "target-features"="+sve" }
----------------
Should I append additional test cases for this test file ?
It seems that the original test file -sve-fixed-length-bitselect.ll- tests specific case (for specific size).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D136858/new/
https://reviews.llvm.org/D136858
More information about the llvm-commits
mailing list