[PATCH] D59952: [VPLAN] Use computed VF for stress testing.

Diego Caballero via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 29 09:56:00 PDT 2019


dcaballe added a comment.

In D59952#1447695 <https://reviews.llvm.org/D59952#1447695>, @fpetrogalli wrote:

> In D59952#1447146 <https://reviews.llvm.org/D59952#1447146>, @dcaballe wrote:
>
> > Should we add a small lit test that covers the new expected behavior? Probably reusing one from D57598 <https://reviews.llvm.org/D57598> with the stress testing flag would suffice.
>
>
> Good idea. Something that invokes stress testing and sets VF = 4 when the computed VF is 1. Did I get it right?


For example. Or one where VF != 4 (computed by `determineVPlanVF`).


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D59952/new/

https://reviews.llvm.org/D59952





More information about the llvm-commits mailing list