[PATCH] D31583: StackColoring: smarter check for slot overlap

[Than McIntosh via llvm-commits]

Yes, but it would have to wait until next week, I will be traveling shortly
and without a good internet connection. If you want to see if submitted
earlier probably best to ask someone else.

On Thu, Jun 8, 2017 at 8:33 AM, Ariel Ben-Yehuda <ariel.byd at gmail.com>
wrote:

> Could you commit the change for me? I don't have commit access myself.
>
> On Thu, Jun 8, 2017 at 3:32 PM, Than McIntosh via Phabricator
> <reviews at reviews.llvm.org> wrote:
> > thanm accepted this revision.
> > thanm added a comment.
> > This revision is now accepted and ready to land.
> >
> > LGTM.
> >
> > FWIW I tried a self-build of clang with and without the change and
> looked at the -stats output to see what sort of improvement there is. Looks
> pretty modest (<1%) but it is positive overall.
> >
> >
> > https://reviews.llvm.org/D31583
> >
> >
> >
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170608/9a3e5e35/attachment.html>