[PATCH] D88460: Strlen loop idiom recognition

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


anjankgk added a comment.

In D88460#2342937 <https://reviews.llvm.org/D88460#2342937>, @hiraditya wrote:

> It'll be nice to know if this patch worked fine with a set of regression tests.
> results on LLVM testsuite, or clang bootstrap will really help.
>
> Thanks for working on this.

Thank you for the feedback @hiraditya! I have tested the latest patch at various options (O2 <https://reviews.llvm.org/owners/package/2/>, O3 <https://reviews.llvm.org/owners/package/3/>, LTO and PGO) against the SPEC benchmarks as well as run the lnt tests and bootstrap tests successfully. If you were concerned about any other specific testsuite, pls let me know.


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

https://reviews.llvm.org/D88460



More information about the llvm-commits mailing list