[PATCH] D34458: [TTI] Refine the cost of EXT in getUserCost()

Evgeny Astigeevich via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 20 05:02:43 PDT 2017


eastig added a comment.

In https://reviews.llvm.org/D34458#815909, @delena wrote:

> I assume this patch should be reverted and reviewed again.


Hi Elena,

Could you please share how significant the regressions are? What benchmarks? Any improvements? Any analysis of the regressions? What are differences in IR?

I have examples when regressions happened not because of the original changes but because other passes had bugs or did not support new cases introduced by the changes.

Thanks,
Evgeny


Repository:
  rL LLVM

https://reviews.llvm.org/D34458





More information about the llvm-commits mailing list