[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