[PATCH] D59952: [VPLAN] Remove option for stress testing.
Francesco Petrogalli via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Mar 28 13:12:56 PDT 2019
fpetrogalli added a comment.
Hi all,
by the comment so far, I think we can abandon this change-set, because:
1. As all of you have pointed out, we still need to do stress testing.
2. Forcing VF=4 for stress testing seems to be a better choice than forcing VF=4 only when the computed VF =1, as it seems more reliable (stress test output doesn't change if we update the bits that compute the VF.
If you agree, I will abandon this patch.
Francesco
Repository:
rL LLVM
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D59952/new/
https://reviews.llvm.org/D59952
More information about the llvm-commits
mailing list