[llvm] r258103 - parseArch() supports more variations of arch names for PowerPC builds

[Hal Finkel via llvm-commits]

----- Original Message -----
> From: "Ismail Donmez" <ismail at i10z.com>
> To: "Hans Wennborg" <hans at chromium.org>
> Cc: "Hal Finkel" <hfinkel at anl.gov>, "Kelvin Li" <kkwli0 at gmail.com>, "llvm-commits" <llvm-commits at lists.llvm.org>
> Sent: Friday, February 12, 2016 5:27:36 AM
> Subject: Re: [llvm] r258103 - parseArch() supports more variations of arch names for PowerPC builds
> 
> ping? This really should be in 3.8 release.

Indeed. I agree.

 -Hal

> 
> On Fri, Feb 5, 2016 at 6:43 PM, Hans Wennborg <hans at chromium.org>
> wrote:
> > Hal: ping?
> >
> > On Fri, Feb 5, 2016 at 3:01 AM, Ismail Donmez <ismail at i10z.com>
> > wrote:
> >> Confirmed that this also fix our SystemZ build failures related to
> >> PR26173.
> >>
> >> On Wed, Feb 3, 2016 at 11:34 PM, Hans Wennborg <hans at chromium.org>
> >> wrote:
> >>> Hal: OK for 3.8?
> >>>
> >>> On Wed, Feb 3, 2016 at 9:22 AM, Ismail Donmez <ismail at i10z.com>
> >>> wrote:
> >>>> Hi,
> >>>>
> >>>> This should fix PR26173 , can we get a 3.8 backport please?
> >>>>
> >>>> On Tue, Jan 19, 2016 at 2:04 AM, Kelvin Li via llvm-commits
> >>>> <llvm-commits at lists.llvm.org> wrote:
> >>>>> Author: kli
> >>>>> Date: Mon Jan 18 18:04:41 2016
> >>>>> New Revision: 258103
> >>>>>
> >>>>> URL: http://llvm.org/viewvc/llvm-project?rev=258103&view=rev
> >>>>> Log:
> >>>>> parseArch() supports more variations of arch names for PowerPC
> >>>>> builds
> >>>>>
> >>>>> Modified:
> >>>>>     llvm/trunk/lib/Support/Triple.cpp
> >>>>>
> >>>>> Modified: llvm/trunk/lib/Support/Triple.cpp
> >>>>> URL:
> >>>>> http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Support/Triple.cpp?rev=258103&r1=258102&r2=258103&view=diff
> >>>>> ==============================================================================
> >>>>> --- llvm/trunk/lib/Support/Triple.cpp (original)
> >>>>> +++ llvm/trunk/lib/Support/Triple.cpp Mon Jan 18 18:04:41 2016
> >>>>> @@ -338,9 +338,9 @@ static Triple::ArchType parseArch(String
> >>>>>      // FIXME: Do we need to support these?
> >>>>>      .Cases("i786", "i886", "i986", Triple::x86)
> >>>>>      .Cases("amd64", "x86_64", "x86_64h", Triple::x86_64)
> >>>>> -    .Case("powerpc", Triple::ppc)
> >>>>> -    .Cases("powerpc64", "ppu", Triple::ppc64)
> >>>>> -    .Case("powerpc64le", Triple::ppc64le)
> >>>>> +    .Cases("powerpc", "ppc32", Triple::ppc)
> >>>>> +    .Cases("powerpc64", "ppu", "ppc64", Triple::ppc64)
> >>>>> +    .Cases("powerpc64le", "ppc64le", Triple::ppc64le)
> >>>>>      .Case("xscale", Triple::arm)
> >>>>>      .Case("xscaleeb", Triple::armeb)
> >>>>>      .Case("aarch64", Triple::aarch64)
> >>>>> @@ -359,7 +359,7 @@ static Triple::ArchType parseArch(String
> >>>>>      .Case("r600", Triple::r600)
> >>>>>      .Case("amdgcn", Triple::amdgcn)
> >>>>>      .Case("hexagon", Triple::hexagon)
> >>>>> -    .Case("s390x", Triple::systemz)
> >>>>> +    .Cases("s390x", "systemz", Triple::systemz)
> >>>>>      .Case("sparc", Triple::sparc)
> >>>>>      .Case("sparcel", Triple::sparcel)
> >>>>>      .Cases("sparcv9", "sparc64", Triple::sparcv9)
> >>>>>
> >>>>>
> >>>>> _______________________________________________
> >>>>> llvm-commits mailing list
> >>>>> llvm-commits at lists.llvm.org
> >>>>> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
> 

-- 
Hal Finkel
Assistant Computational Scientist
Leadership Computing Facility
Argonne National Laboratory