----- Original Message ----- > From: "Renato Golin" <renato.go...@linaro.org> > To: "Hans Wennborg" <h...@chromium.org> > Cc: "llvm-dev" <llvm-...@lists.llvm.org>, "cfe-dev" <cfe-...@lists.llvm.org>, > "LLDB Dev" <lldb-dev@lists.llvm.org>, > "openmp-dev (openmp-...@lists.llvm.org)" <openmp-...@lists.llvm.org>, > release-test...@lists.llvm.org, "Cong Hou" > <co...@google.com>, "Davide Italiano" <dav...@freebsd.org>, "Quentin > Colombet" <qcolom...@apple.com>, "Hal Finkel" > <hfin...@anl.gov>, "Chandler Carruth" <chandl...@google.com>, "JF Bastien" > <j...@google.com>, "Nemanja Ivanovic" > <nemanja.i....@gmail.com>, "Matthias Braun" <ma...@braunis.de> > 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 > <lldb-dev@lists.llvm.org> 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
