[PATCH] D100258: [VPlan] Add first VPlan version of sinkScalarOperands.

[Florian Hahn via Phabricator via llvm-commits]

fhahn added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp:127
+    // All users of SinkCandidate must be in the same block in order to perform
+    // sinking. Therefor the destination block for sinking must match the block
+    // containing the first user.
----------------
Ayal wrote:
> "Therefor the" >> "Therefore the"
Ah yes! Fixed in the committed version, thanks!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100258