[PATCH] D53349: [VPlan] Changes to implement VPlan based predication for VPlan-native path.

Satish K Guggilla via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 12 17:46:46 PST 2018


sguggill marked 2 inline comments as done.
sguggill added inline comments.


================
Comment at: lib/Transforms/Vectorize/LoopVectorize.cpp:6980
+  // code. Masked vector code generation support will follow soon.
+  if (VPlanBuildStressTest || EnableVPlanPredication)
     return false;
----------------
rengolin wrote:
> Do you have an RFC or review on the code generation part? It would be good to have an idea soon enough.
I have the code generation changes working in my private workspace. I need to tie up some loose ends before I start the review. These changes make use of changes in D53355 Introducing VPPHINode. I will be out the next two weeks and I will try to get the CG review started once I get back.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D53349





More information about the llvm-commits mailing list