[PATCH] D88382: [VPlan] Turn VPReductionRecipe into a VPValue

Dave Green via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 25 00:19:29 PST 2020


dmgreen added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/VPlan.cpp:99
     return U;
   if (auto *U = dyn_cast<VPWidenMemoryInstructionRecipe>(this))
     return U;
----------------
fhahn wrote:
> now that `VPReductionRecipe` is also a VPUser, it should also be added here. (this will go away once all recipes have been converted)
Yeah thanks, I noticed this one as I was getting the patch together for committing. I was trying a sanitizer build to check they are OK, which is taking a rather long time..


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

https://reviews.llvm.org/D88382



More information about the llvm-commits mailing list