[PATCH] D59952: [VPLAN] Minor improvement to testing and debug messages.

Diego Caballero via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 29 14:39:04 PDT 2019


dcaballe added a comment.

> Or are you saying this needs to be done explicitly for stress testing?

Yes, something that covers the new stress testing behavior that this patch is introducing, to make sure that we preserve it. It could be a test using the stress testing flag and VF!=4 coming from `determinePlanVF` and/or a test using stress testing flag and where the default VF=4 is taken.
We are not generating code in stress testing mode so maybe you can match some VF information from the debug print.
Please, let me know if you find any roadblock.


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

https://reviews.llvm.org/D59952





More information about the llvm-commits mailing list