--===============9188017145348444244== Content-Type: multipart/alternative; boundary=bcaec5196aab8b3cc4051c1ea49e --bcaec5196aab8b3cc4051c1ea49e Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable On Thu, Jul 30, 2015 at 10:03 AM, Ulrich Weigand wrote: > Hi Hans, > > thanks! Just a quick update: it turns out the sanitizer build bots > uncovered an uninitialized variable access. I've now fixed this in > r243644, and the build bot is green again. So if this feature is > backported, please add that revision as well. > > Sorry for the extra confusion ... > > Bye, > Ulrich > > hwennborg@google.com wrote on 30.07.2015 17:02:32: > > > Hi Ulrich, > > > > I'm happy to merge if Richard gives the owner's approval. > It's a bit unusual to take a whole new feature at this stage in the release, but if you're happy with it, I think it's a suitably low-risk patch (the biggest risk is that the feature itself would not work in some way), so go ahead. > > Cheers, > > Hans > > > > On Thu, Jul 30, 2015 at 7:24 AM, Ulrich Weigand > > wrote: > > > > > > Hi Hans, hi Richard, > > > > > > I've just checked support for the System z vector language extensions > (and > > > associated intrinsics header file) into clang mainline as revisions > r243642 > > > and r243643. > > > > > > This is the last missing part to complete support for the IBM z13 > mainframe > > > architecture in LLVM and clang at a level compatible with current GCC > > > mainline. All the other parts (LLVM back-end support, and basic clan= g > > > support for -march=3Dz13 except for the language extension) are alrea= dy > in > > > 3.7 since they went in before the release branched. The language > extension > > > however is not in the branch. > > > > > > In order for LLVM 3.7 to be able to claim full z13 support, it would = be > > > really good if that last missing feature could still make it into the > > > branch. I'm aware that we're already quite late in the cycle -- do y= ou > > > still see a chance to get it in? If there's anything I can to do hel= p > with > > > this, please let me know. Thanks! > > > > > > > > > Mit freundlichen Gruessen / Best Regards > > > > > > Ulrich Weigand > > > > > > -- > > > Dr. Ulrich Weigand | Phone: +49-7031/16-3727 > > > STSM, GNU/Linux compilers and toolchain > > > IBM Deutschland Research & Development GmbH > > > Vorsitzende des Aufsichtsrats: Martina Koederitz | Gesch=C3=A4ftsf= =C3=BChrung: > Dirk > > > Wittkopp > > > Sitz der Gesellschaft: B=C3=B6blingen | Registergericht: Amtsgerich= t > > > Stuttgart, HRB 243294 > > > > > > > --bcaec5196aab8b3cc4051c1ea49e Content-Type: text/html; charset=UTF-8 Content-Transfer-Encoding: quoted-printable
On T= hu, Jul 30, 2015 at 10:03 AM, Ulrich Weigand <Ulrich.Weigand@de.i= bm.com> wrote:
Hi Hans,

thanks!=C2=A0 =C2=A0Just a quick update: it turns out the sanitizer build b= ots
uncovered an uninitialized variable access.=C2=A0 I've now fixed this i= n
r243644, and the build bot is green again.=C2=A0 So if this feature is
backported, please add that revision as well.

Sorry for the extra confusion ...

Bye,
Ulrich

hwennborg@google.com wrote on 3= 0.07.2015 17:02:32:

> Hi Ulrich,
>
> I'm happy to merge if Richard gives the owner's approval.
<= /div>

It's a bit unusual to take = a whole new feature at this stage in the release, but if you're happy w= ith it, I think it's a suitably low-risk patch (the biggest risk is tha= t the feature itself would not work in some way), so go ahead.
= =C2=A0
> Cheers,
> Hans
>
> On Thu, Jul 30, 2015 at 7:24 AM, Ulrich Weigand
> <Ulrich.Weigand@de.ibm= .com> wrote:
> >
> > Hi Hans, hi Richard,
> >
> > I've just checked support for the System z vector language ex= tensions
(and
> > associated intrinsics header file) into clang mainline as revisio= ns
r243642
> > and r243643.
> >
> > This is the last missing part to complete support for the IBM z13=
mainframe
> > architecture in LLVM and clang at a level compatible with current= GCC
> > mainline.=C2=A0 All the other parts (LLVM back-end support, and b= asic clang
> > support for -march=3Dz13 except for the language extension) are a= lready
in
> > 3.7 since they went in before the release branched.=C2=A0 The lan= guage
extension
> > however is not in the branch.
> >
> > In order for LLVM 3.7 to be able to claim full z13 support, it wo= uld be
> > really good if that last missing feature could still make it into= the
> > branch.=C2=A0 I'm aware that we're already quite late in = the cycle -- do you
> > still see a chance to get it in?=C2=A0 If there's anything I = can to do help
with
> > this, please let me know.=C2=A0 =C2=A0Thanks!
> >
> >
> > Mit freundlichen Gruessen / Best Regards
> >
> > Ulrich Weigand
> >
> > --
> >=C2=A0 =C2=A0Dr. Ulrich Weigand | Phone: +49-7031/16-3727
> >=C2=A0 =C2=A0STSM, GNU/Linux compilers and toolchain
> >=C2=A0 =C2=A0IBM Deutschland Research & Development GmbH
> >=C2=A0 =C2=A0Vorsitzende des Aufsichtsrats: Martina Koederitz | Ge= sch=C3=A4ftsf=C3=BChrung:
Dirk
> > Wittkopp
> >=C2=A0 =C2=A0Sitz der Gesellschaft: B=C3=B6blingen | Registergeric= ht: Amtsgericht
> > Stuttgart, HRB 243294
> >
>


--bcaec5196aab8b3cc4051c1ea49e-- --===============9188017145348444244== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ cfe-dev mailing list cfe-dev@cs.uiuc.edu http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev --===============9188017145348444244==--