[PATCH] D75233: [LoopTerminology] LCSSA Form

Stefanos Baziotis via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 19 18:40:03 PDT 2020


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

- Added other advantages.
- Rephrased Chris's wording a little bit as it was not very clear to me. Also, the deep analysis, the more I thought of it and as @Meinersbur mentioned,

does not seem to be needed any way. We could use `RAUW()` to achieve this effect (although it would still be more expensive).

2 questions:

1. Is it "loop closed PHI node" or "loop closing PHI node"?
2. Does LCSSA pass have to do this deep analysis? Because to me, it seems it can just use `RAUW`?


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

https://reviews.llvm.org/D75233

Files:
  llvm/docs/LoopTerminology.rst
  llvm/docs/Passes.rst

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D75233.251522.patch
Type: text/x-patch
Size: 5594 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200320/669c2c45/attachment.bin>


More information about the llvm-commits mailing list