[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