[PATCH] D43079: [TTI CostModel] change default cost of FP ops to 1 (PR36280)

Wed Feb 21 10:15:25 PST 2018

anemet added a comment.

In https://reviews.llvm.org/D43079#1013927, @spatel wrote:

> In https://reviews.llvm.org/D43079#1013907, @anemet wrote:
>
> > Seeing such major swings, my preference would be to revert and put the new version up for review (I think that your hack works).  Then commit the new combined version after a few days so that the perf bots got a chance to recover.  What do you think?
>
>
> Ok, this was too ambitious. Reverted at https://reviews.llvm.org/rL325658 and reopened: 
>  https://bugs.llvm.org/show_bug.cgi?id=36280

Thanks!  I can confirm that our perf numbers have recovered.

Repository:
  rL LLVM

https://reviews.llvm.org/D43079