[PATCH] D32563: Add LiveRangeShrink pass to shrink live range within BB.

Matthias Braun via llvm-commits llvm-commits at lists.llvm.org
Wed May 17 13:06:18 PDT 2017


> On May 17, 2017, at 1:02 PM, Chandler Carruth via llvm-commits <llvm-commits at lists.llvm.org> wrote:
> 
> On Wed, May 17, 2017 at 12:09 PM Daniel Berlin <dberlin at dberlin.org <mailto:dberlin at dberlin.org>> wrote:
> Just to note: they weren't raised in review, they are being raised based on the performance regression discovered :)
> 
> Ah good to know, I misunderstood Matthias's email.
Yes they were discovered after the initial review and from a CPU target perspective the pass didn't seem to hurt.

I answered here because I think it is convenient to have the discussion of a patch with the initial review instead of the commit mail.

- Matthias
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170517/14fbba35/attachment-0001.html>


More information about the llvm-commits mailing list