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

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 20 17:49:29 PST 2018


spatel reopened this revision.
spatel added a comment.
This revision is now accepted and ready to land.

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


Repository:
  rL LLVM

https://reviews.llvm.org/D43079





More information about the llvm-commits mailing list