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

[Hans Wennborg via llvm-commits]

Merged in r261015.

Thanks,
Hans

On Tue, Feb 16, 2016 at 1:49 PM, Hal Finkel <hfinkel at anl.gov> wrote:
> ----- 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