Re: [lldb-dev] [3.8 Release] Release status

2016-02-20 Thread Davide Italiano via lldb-dev
On Thu, Feb 18, 2016 at 4:22 PM, Hans Wennborg  wrote:
> According to the schedule (e.g. on the right on llvm.org), we should
> have tagged the release by now, but we haven't, so we're officially
> behind schedule. I'm still optimistic that we can wrap this up pretty
> soon, though.
>
> This is what's blocking us:
>
> - PR26509: Crash in InnerLoopVectorizer::vectorizeLoop()
>   I'm waiting to hear what Cong comes up with, otherwise we can revert
> r255691 on the branch
>
> - Shrink-wrapping vs TLS: Davide and Quentin are working on it
>

http://reviews.llvm.org/D17427

--
Davide
___
lldb-dev mailing list
lldb-dev@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-dev


Re: [lldb-dev] [llvm-dev] [3.8 Release] Release status

2016-02-20 Thread John Kåre Alsaker via lldb-dev
I'd suggest that https://llvm.org/bugs/show_bug.cgi?id=26554 should be
on that list.

- John Kåre

On Fri, Feb 19, 2016 at 1:54 AM, Davide Italiano via llvm-dev
 wrote:
> On Thu, Feb 18, 2016 at 4:22 PM, Hans Wennborg  wrote:
>> According to the schedule (e.g. on the right on llvm.org), we should
>> have tagged the release by now, but we haven't, so we're officially
>> behind schedule. I'm still optimistic that we can wrap this up pretty
>> soon, though.
>>
>> This is what's blocking us:
>>
>> - PR26509: Crash in InnerLoopVectorizer::vectorizeLoop()
>>   I'm waiting to hear what Cong comes up with, otherwise we can revert
>> r255691 on the branch
>>
>> - Shrink-wrapping vs TLS: Davide and Quentin are working on it
>>
>
> http://reviews.llvm.org/D17427
>
> --
> Davide
> ___
> LLVM Developers mailing list
> llvm-...@lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev
___
lldb-dev mailing list
lldb-dev@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-dev


Re: [lldb-dev] [3.8 Release] Release status

2016-02-20 Thread Hal Finkel via lldb-dev
- Original Message -
> From: "Renato Golin" 
> To: "Hans Wennborg" 
> Cc: "llvm-dev" , "cfe-dev" , 
> "LLDB Dev" ,
> "openmp-dev (openmp-...@lists.llvm.org)" , 
> release-test...@lists.llvm.org, "Cong Hou"
> , "Davide Italiano" , "Quentin 
> Colombet" , "Hal Finkel"
> , "Chandler Carruth" , "JF Bastien" 
> , "Nemanja Ivanovic"
> , "Matthias Braun" 
> Sent: Friday, February 19, 2016 9:33:16 AM
> Subject: Re: [lldb-dev] [3.8 Release] Release status
> 
> On 19 February 2016 at 00:22, Hans Wennborg via lldb-dev
>  wrote:
> > - PR26600: Loop vectorization creates an unsafe out-of-bounds load
> >   There's a patch out: http://reviews.llvm.org/D17332
> >   But no comments yet.. Hal?
> 
> This looks like a serious bug but Hal's idea is still unproven.

I feel your statement misrepresents my review of the patch. What I said in the 
review was:

[from review]
How complicated would it be, instead of bailing out when we have a group 
without the last member, to peel off the last vector iteration instead (i.e. 
jump to the scalar tail loop one vector-loop iteration "early")?

It seems like that would be a better solution (although, if you agree, but it 
seems too complicated to implement for the release branch, I'm fine with taking 
this (and pulling it into the release branch), and then implementing the better 
solution in trunk only).
[end from review]

What we have now is one easy fix, and I simply asked if we could get a much 
better easy fix in short order (i.e. for 3.8.0). Regardless, I feel I was quite 
clear in the review that this fix is otherwise fine, and we definitely need 
something for 3.8.0. It is a serious bug.

Thanks again,
Hal


> I
> think we should land the patch as it is, since it disables the bad
> behaviour, and think about a fix later. Depending on how complex the
> fix is, we might not even merge it into 3.8.x later on, but we need
> the fix in 3.8.0.
> 
> 
> > - ARM: fix VFP asm constraints: http://reviews.llvm.org/D17349
> >   It's not a regression, but I'll take it if it lands real soon.
> 
> No brainer. Accepted on trunk by compnerd, I'll merge into release_38
> shortly.
> 
> cheers,
> --renato
> 

-- 
Hal Finkel
Assistant Computational Scientist
Leadership Computing Facility
Argonne National Laboratory
___
lldb-dev mailing list
lldb-dev@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-dev