[PATCH] D88460: Strlen loop idiom recognition

Thu Dec 17 23:14:22 PST 2020

hiraditya added a comment.

In D88460#2458205 <https://reviews.llvm.org/D88460#2458205>, @anjankgk wrote:

> 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, O3, 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.

Nice, if we have performance improvements on any of the benchmarks/workloads then do share.

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

https://reviews.llvm.org/D88460