[PATCH] D12334: [ARM] Do not use vtrn for vectorshuffle if the order is reversed

Ahmed Bougacha via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 3 15:48:38 PDT 2015


ab requested changes to this revision.
ab added a comment.
This revision now requires changes to proceed.

No: the added tests that generate <8 x i32> aren't testing the patched code: <8 x i32> isn't legal on ARM, so the tests' shufflevectors will be split into 2 separate <4 x i32> shufflevectors. Please test using (for instance) <4 x i16> and <8 x i16> instead of <4 x i32> and <8 x i32>.


http://reviews.llvm.org/D12334





More information about the llvm-commits mailing list