[prev in list] [next in list] [prev in thread] [next in thread]
List: cfe-dev
Subject: Re: [cfe-dev] System z vector language extensions for 3.7 branch?
From: Richard Smith <richard () metafoo ! co ! uk>
Date: 2015-07-30 21:46:58
Message-ID: CAOfiQq=-J4GZhtR00YsfzUozTirkHy=W7-f4R+MyNCSsC+sQNA () mail ! gmail ! com
[Download RAW message or body]
[Attachment #2 (multipart/alternative)]
On Thu, Jul 30, 2015 at 10:03 AM, Ulrich Weigand <Ulrich.Weigand@de.ibm.com>
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
> > <Ulrich.Weigand@de.ibm.com> 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 clang
> > > support for -march=z13 except for the language extension) are already
> 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 you
> > > still see a chance to get it in? If there's anything I can to do help
> 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äftsführung:
> Dirk
> > > Wittkopp
> > > Sitz der Gesellschaft: Böblingen | Registergericht: Amtsgericht
> > > Stuttgart, HRB 243294
> > >
> >
>
>
[Attachment #5 (text/html)]
<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Jul 30, 2015 \
at 10:03 AM, Ulrich Weigand <span dir="ltr"><<a \
href="mailto:Ulrich.Weigand@de.ibm.com" \
target="_blank">Ulrich.Weigand@de.ibm.com</a>></span> wrote:<br><blockquote \
class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc \
solid;padding-left:1ex">Hi Hans,<br> <br>
thanks! Just a quick update: it turns out the sanitizer build bots<br>
uncovered an uninitialized variable access. I've now fixed this in<br>
r243644, and the build bot is green again. So if this feature is<br>
backported, please add that revision as well.<br>
<br>
Sorry for the extra confusion ...<br>
<br>
Bye,<br>
Ulrich<br>
<div class="HOEnZb"><div class="h5"><br>
<a href="mailto:hwennborg@google.com">hwennborg@google.com</a> wrote on 30.07.2015 \
17:02:32:<br> <br>
> Hi Ulrich,<br>
><br>
> I'm happy to merge if Richard gives the owner's \
approval.<br></div></div></blockquote><div><br></div><div>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.</div><div> </div><blockquote \
class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc \
solid;padding-left:1ex"><div class="HOEnZb"><div class="h5"> > Cheers,<br>
> Hans<br>
><br>
> On Thu, Jul 30, 2015 at 7:24 AM, Ulrich Weigand<br>
> <<a href="mailto:Ulrich.Weigand@de.ibm.com">Ulrich.Weigand@de.ibm.com</a>> \
wrote:<br> > ><br>
> > Hi Hans, hi Richard,<br>
> ><br>
> > I've just checked support for the System z vector language \
extensions<br> (and<br>
> > associated intrinsics header file) into clang mainline as revisions<br>
r243642<br>
> > and r243643.<br>
> ><br>
> > This is the last missing part to complete support for the IBM z13<br>
mainframe<br>
> > architecture in LLVM and clang at a level compatible with current GCC<br>
> > mainline. All the other parts (LLVM back-end support, and basic clang<br>
> > support for -march=z13 except for the language extension) are already<br>
in<br>
> > 3.7 since they went in before the release branched. The language<br>
extension<br>
> > however is not in the branch.<br>
> ><br>
> > In order for LLVM 3.7 to be able to claim full z13 support, it would be<br>
> > really good if that last missing feature could still make it into the<br>
> > branch. I'm aware that we're already quite late in the cycle -- \
do you<br> > > still see a chance to get it in? If there's anything I can \
to do help<br> with<br>
> > this, please let me know. Thanks!<br>
> ><br>
> ><br>
> > Mit freundlichen Gruessen / Best Regards<br>
> ><br>
> > Ulrich Weigand<br>
> ><br>
> > --<br>
> > Dr. Ulrich Weigand | Phone: +49-7031/16-3727<br>
> > STSM, GNU/Linux compilers and toolchain<br>
> > IBM Deutschland Research & Development GmbH<br>
> > Vorsitzende des Aufsichtsrats: Martina Koederitz | \
Geschäftsführung:<br> Dirk<br>
> > Wittkopp<br>
> > Sitz der Gesellschaft: Böblingen | Registergericht: Amtsgericht<br>
> > Stuttgart, HRB 243294<br>
> ><br>
><br>
<br>
</div></div></blockquote></div><br></div></div>
_______________________________________________
cfe-dev mailing list
cfe-dev@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic