[PATCH] D29862: LSR: an alternative way to resolve complex solution

[Dave Green via Phabricator via llvm-commits]

dmgreen added a comment.

Hello,

Looks like you committed with the options set to true. This seems to introduce some regressions we are seeing in internal benchmarks. While there are some good improvements too, some benchmarks are regressing 15-20%. For example I believe these Shootout-C++ matrix regressions are due to this commit:
http://llvm.org/perf/db_default/v4/nts/daily_report/2017/2/21?day_start=16

Unless you have any objections, I think we should set the option default to false for the time being, like the review says.

Thanks,
Dave


Repository:
  rL LLVM

https://reviews.llvm.org/D29862