[PATCH] D59952: [VPLAN] Remove option for stress testing.

Diego Caballero via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 28 13:53:31 PDT 2019


dcaballe added a comment.

> I thought the point was to just have a VF to build a VPlan with and as it currently stands we build the same VPlans with VF = 4 or any other VF I think. With the programmatically determined VF, we would be a little closer to reality in the stress testing. As Hideki mentioned, having a constant factor for stress testing might have other benefits later on though

Yes, that sounds good to me. In the worst case, we could always use stress testing + user VF to achieve that.


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