[PATCH] D75013: [LoopTerminology] Rotated Loops

Stefanos Baziotis via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 10 19:09:51 PDT 2020


baziotis updated this revision to Diff 249537.
baziotis added a comment.

- Changed loop-rotate to LoopRotate as the latter is more linked with the pass while the former seems like a cmd arguments.
- Clarification on the hoisting of instructions.
- Removed the reference to LoopSimplify. I think it is better. I tried to write it a couple of times it introduces duplicate doc. i.e. LoopRotate is a LoopPass and so LoopSimplify is ran before it, but this is mentioned in the doc about Loop Simplify Form.


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

https://reviews.llvm.org/D75013

Files:
  llvm/docs/LoopTerminology.rst
  llvm/docs/Passes.rst
  llvm/docs/loop-terminology-guarded-loop.png
  llvm/docs/loop-terminology-initial-loop.png
  llvm/docs/loop-terminology-rotated-loop.png

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D75013.249537.patch
Type: text/x-patch
Size: 7046 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200311/7bc08b6f/attachment.bin>


More information about the llvm-commits mailing list