[PATCH] D41851: X86 Tests: Update more isel tests with FastVariableShuffle feature

Zvi Rackover via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 9 05:04:15 PST 2018


zvi added a comment.

In https://reviews.llvm.org/D41851#970876, @RKSimon wrote:

>   Assuming D41436 is accepted, is the plan to remove the +fast-variable-shuffle arg from the avx512 cases? In which case might it make sense to commit the avx2 and avx512 changes separately? 


IMO we should still have the flag enabled in the AVX512 tests to ensure that we are testing a configuration that resembles the processor it intends to cover.


https://reviews.llvm.org/D41851





More information about the llvm-commits mailing list