[PATCH] D84683: [VPlan] Use VPValue def for VPWidenGEPRecipe.

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 6 01:49:56 PDT 2020


fhahn added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/VPlan.cpp:127-128
     return V;
+  if (auto *V = dyn_cast<VPWidenGEPRecipe>(this))
+    return V;
   return nullptr;
----------------
dmgreen wrote:
> Lets try and remove this again sooner rather than later ;)
Yeah, `toVPValue` and `toVPUser` are just convenient for the transition period. They can be removed as soon as the transition is complete (D88379, D88378)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84683



More information about the llvm-commits mailing list