<p dir="ltr">It's larger than I'd like for a merge onto the branch, but yes, it looks OK.</p>
<div class="gmail_extra"><br><div class="gmail_quote">On 7 Jun 2016 9:20 a.m., "Tom Stellard via cfe-commits" <<a href="mailto:cfe-commits@lists.llvm.org">cfe-commits@lists.llvm.org</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Richard,<br>
<br>
Is this OK for the 3.8 branch?<br>
<br>
<a href="http://reviews.llvm.org/rL259776" rel="noreferrer" target="_blank">reviews.llvm.org/rL259776</a><br>
<br>
-Tom<br>
<br>
On Mon, May 30, 2016 at 11:35:04AM +0000, Hahnfeld, Jonas wrote:<br>
> Assuming I got it right...<br>
><br>
> > -----Original Message-----<br>
> > From: Alexey Bataev [mailto:<a href="mailto:a.bataev@hotmail.com">a.bataev@hotmail.com</a>]<br>
> > Sent: Monday, May 30, 2016 1:31 PM<br>
> > To: Hahnfeld, Jonas<br>
> > Cc: <a href="mailto:cfe-commits@lists.llvm.org">cfe-commits@lists.llvm.org</a>; Tom Stellard<br>
> > Subject: Re: 3.8 Merge Request: r259776<br>
> ><br>
> > I can try to rebase it to 3.8.1 but Tom wants an approve from Richard.<br>
> > Ping him!<br>
> ><br>
> > Best regards,<br>
> > Alexey Bataev<br>
> > =============<br>
> > Software Engineer<br>
> > Intel Compiler Team<br>
> ><br>
> ><br>
> ><br>
> > 30.05.2016 14:19, Hahnfeld, Jonas пишет:<br>
> > > Hi,<br>
> > ><br>
> > > last possible ping as changes are required to be merged by tomorrow...<br>
> > ><br>
> > > Cheers,<br>
> > > Jonas<br>
> > ><br>
> > >> -----Original Message-----<br>
> > >> From: llvm-commits [mailto:<a href="mailto:llvm-commits-bounces@lists.llvm.org">llvm-commits-bounces@lists.llvm.org</a>] On<br>
> > >> Behalf Of Hahnfeld, Jonas via llvm-commits<br>
> > >> Sent: Monday, May 23, 2016 8:13 AM<br>
> > >> To: Tom Stellard<br>
> > >> Cc: Bataev, Alexey (<a href="mailto:a.bataev@hotmail.com">a.bataev@hotmail.com</a>);<br>
> > >> <a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a><br>
> > >> Subject: RE: 3.8 Merge Request: r259776<br>
> > >><br>
> > >> Ping. I would really much like to see this (and possibly other OpenMP<br>
> > > bugs)<br>
> > >> fixed for 3.8.1.<br>
> > >><br>
> > >> Thanks,<br>
> > >> Jonas<br>
> > >><br>
> > >>> -----Original Message-----<br>
> > >>> From: Tom Stellard [mailto:<a href="mailto:tom@stellard.net">tom@stellard.net</a>]<br>
> > >>> Sent: Thursday, May 19, 2016 7:37 PM<br>
> > >>> To: Hahnfeld, Jonas<br>
> > >>> Cc: <a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>; Bataev, Alexey<br>
> > >>> (<a href="mailto:a.bataev@hotmail.com">a.bataev@hotmail.com</a>)<br>
> > >>> Subject: Re: 3.8 Merge Request: r259776<br>
> > >>><br>
> > >>> On Thu, May 19, 2016 at 11:25:43AM +0000, Hahnfeld, Jonas wrote:<br>
> > >>>> This bug also exists in 3.8.0. Alexey already agreed to merge this<br>
> > >>>> back<br>
> > >> in<br>
> > >>>> March:<br>
> > >>>> <a href="http://lists.llvm.org/pipermail/cfe-commits/Week-of-Mon-" rel="noreferrer" target="_blank">http://lists.llvm.org/pipermail/cfe-commits/Week-of-Mon-</a><br>
> > >>> 20160314/152975.html<br>
> > >>>> Alexey, are there other OpenMP commits needed for 3.8.1?<br>
> > >>>><br>
> > >>> Hi,<br>
> > >>><br>
> > >>> I would also like Richard to take a look at this one to.<br>
> > >>><br>
> > >>> Richard, does this look OK to merge:<br>
> > >>><br>
> > >>> <a href="http://reviews.llvm.org/rL259776" rel="noreferrer" target="_blank">reviews.llvm.org/rL259776</a><br>
> > >>><br>
> > >>> -Tom<br>
> > >>><br>
> > >>>> Cheers,<br>
> > >>>> Jonas<br>
> ><br>
><br>
<br>
<br>
_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@lists.llvm.org">cfe-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits</a><br>
</blockquote></div></div>