D29862: LSR: an alternative way to resolve complex solution

David Green via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 2 11:11:54 PST 2017


Hello,

Firstly, thanks for doing this work and looking into the regressions. My comment about changing the default was made in haste before I had read the llvm-dev post, I'll leave that kind of thing to you.

I ran the new patch against some downstream benchmarks and, so long as I am looking at this correctly, the results look pretty positive. These were not full performance runs, but I ran some quick runs on M23 and M7's, and there were some good improvements in the places we previously saw regressions. I'm not sure about the hmmer benchmarks, but I would like to see the results on a full run when the patch makes it in.

Cheers
Dave


More information about the llvm-commits mailing list