[PATCH] D28975: [LV] Introducing VPlan to model the vectorized code and drive its transformation

[Ayal Zaks via Phabricator via llvm-commits]

Ayal added inline comments.


================
Comment at: docs/VectorizationPlan.rst:287
+  instructions that will appear consecutively in a basic block of the vectorized
+  version. The instructions of such a basic block originate from one or more
+  VPBasicBlocks.
----------------
mkuper wrote:
> How can they originate from more than one basic block? Can you be more explicit about the relationship between IR basic blocks and VP basic blocks?
The sentence above meant to emphasize that VPBasicBlocks are not necessarily maximal. A VPBasicBlock can have a single successor VPBasicBlock, which in turn has a single predecessor. Eventually both will contribute their instructions into a common IR basic block of the vectorized version.


https://reviews.llvm.org/D28975