<p dir="ltr">Okay, thanks for the update.</p>
<br><div class="gmail_quote"><div dir="ltr">On Thu, Sep 24, 2015, 10:34 PM Michael Spencer <<a href="mailto:bigcheesegs@gmail.com">bigcheesegs@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Thu, Sep 24, 2015 at 12:12 PM, Eric Christopher <<a href="mailto:echristo@gmail.com" target="_blank">echristo@gmail.com</a>> wrote:<br>
> Hi Michael,<br>
><br>
>><br>
>> > I notice that the -load-combine pass is not being used since its initial<br>
>> > commit in r209791.<br>
>> > What's the status on this? Should it be removed? Are there any plan to<br>
>> > use<br>
>> > it at some point in the future?<br>
>> ><br>
>> > Thanks,<br>
>> > -- Larisse.<br>
>><br>
>> It's waiting for a machine pass to do load slicing to be enabled by<br>
>> default. I've been planning on doing this, but it's not going to<br>
>> happen soon.<br>
>><br>
><br>
> So, what would you like to do then? :)<br>
><br>
> It's been pretty much dead code since it was committed more than a year ago<br>
> and we try to avoid keeping dead code in tree. It's not a ton of work to<br>
> keep it updated, but that's all that's happened to the pass in the mean time<br>
> and we don't have any idea when it might be usable.<br>
><br>
> Thoughts?<br>
><br>
> -eric<br>
<br>
So it actually appears that someone has written a load slicer since<br>
that went in. I'll figure out next week if it fixes the performance<br>
problems I was encountering before and hopefully turn it on. If not<br>
I'm fine with removing it.<br>
<br>
- Michael Spencer<br>
</blockquote></div>