[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