[PATCH] D80219: [VPlan] Use VPUser for VPWidenSelectRecipe operands (NFC).

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun May 24 09:36:27 PDT 2020


fhahn marked an inline comment as done.
fhahn added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/LoopVectorize.cpp:4438
     Value *Sel =
         Builder.CreateSelect(InvariantCond ? ScalarCond : Cond, Op0, Op1);
     VectorLoopValueMap.setVectorValue(&I, Part, Sel);
----------------
Ayal wrote:
> As noted earlier, would be good to avoid getting both ScalarCond and Cond, only to use one of them. Can be fixed separately.
Ah right! I've pushed 0deab8a54fd8d83853740eb751ddba967ad514f7


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80219





More information about the llvm-commits mailing list