<p dir="ltr">Yes. That's reasonable. We should only avoid adding rL266217 without also rL266438.</p>
<p dir="ltr">Nirav</p>
<div class="gmail_quote">On May 18, 2016 5:17 PM, "Tom Stellard" <<a href="mailto:tom@stellard.net">tom@stellard.net</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Nirav,<br>
<br>
Does it make sense to merge <a href="http://reviews.llvm.org/rL266438" rel="noreferrer" target="_blank">http://reviews.llvm.org/rL266438</a> even<br>
without  <a href="http://reviews.llvm.org/rL266217" rel="noreferrer" target="_blank">http://reviews.llvm.org/rL266217</a>.<br>
<br>
-Tom<br>
<br>
On Wed, May 18, 2016 at 05:03:10PM -0400, Nirav Davé wrote:<br>
> This patch causes building clang for ppc64le to trip over a latent bug<br>
> (PR27350) which was fixed in <a href="http://reviews.llvm.org/rL266438" rel="noreferrer" target="_blank">http://reviews.llvm.org/rL266438</a>. Both or<br>
> neither should be applied.<br>
><br>
> -Nirav<br>
><br>
> On Wed, May 18, 2016 at 4:45 PM, Tom Stellard <<a href="mailto:tom@stellard.net">tom@stellard.net</a>> wrote:<br>
><br>
> > Hi Owen,<br>
> ><br>
> > Is this OK to merge to the 3.8 branch: <a href="http://reviews.llvm.org/rL266217" rel="noreferrer" target="_blank">http://reviews.llvm.org/rL266217</a><br>
> ><br>
> > Thanks,<br>
> > Tom<br>
> ><br>
> > On Tue, May 10, 2016 at 06:53:10PM +0000, Kit Barton wrote:<br>
> > >    Hi,<br>
> > >    Could we please include rL266217 in the 3.8.1 release?<br>
> > >    It fixes PR27138, in addition to PR26827 (which was noted in the<br>
> > >    Phabricator review).<br>
> > ><br>
> > >    Thanks,<br>
> > ><br>
> > >    Kit Barton, Ph.D.<br>
> > >    LLVM Development on POWER<br>
> > >    IBM Toronto Lab, D2/929/8200/MKM<br>
> > >    8200 Warden Ave, Markham, L6G 1C7<br>
> > >    (905) 413-3452<br>
> > >    <a href="mailto:kbarton@ca.ibm.com">kbarton@ca.ibm.com</a><br>
> ><br>
</blockquote></div>