[PATCH] D88460: Strlen loop idiom recognition

Anjan Kumar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 16 09:12:39 PST 2020


anjankgk updated this revision to Diff 312230.
anjankgk added a comment.

Changes from previous review comments:

- Replaced LCSSAEv->getStart() by LoadEv->getStart()
- Added another test function where multiple LCSSAPhi nodes were present and therefore shouldn't have replaced with strlen function.


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

https://reviews.llvm.org/D88460

Files:
  llvm/lib/Transforms/Scalar/LoopIdiomRecognize.cpp
  llvm/test/Transforms/LoopIdiom/recognize-strlen.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D88460.312230.patch
Type: text/x-patch
Size: 14383 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20201216/d87cd540/attachment.bin>


More information about the llvm-commits mailing list