r177161 - Take in account the triplet 'powerpc-linux-gnuspe' for PowerPC SPE. Done for the port of Debian on this arch. More information on: http://wiki.debian.org/PowerPCSPEPort Patch by Roland Stigge

[Jordan Rose]

>From that site:

> The 'powerpcspe' architecture is a binary-incompatible variant of the PowerPC/POWER designed and supported by FreeScale and IBM. It is also known under the trade names "e500"/"MPC8500" and "e200"/"MPC5xx".

I'm not really qualified to make this call, but it sounds like changing the OS/vendor part of the triple seems like the wrong thing to do. Is there a reason why you didn't change the CPU part and then just make it an alias of PowerPC for now?

Jordan


On Mar 15, 2013, at 9:22 , Sylvestre Ledru <sylvestre at debian.org> wrote:

> Author: sylvestre
> Date: Fri Mar 15 11:22:43 2013
> New Revision: 177161
> 
> URL: http://llvm.org/viewvc/llvm-project?rev=177161&view=rev
> Log:
> Take in account the triplet 'powerpc-linux-gnuspe' for PowerPC SPE. Done for the port of Debian on this arch. More information on: http://wiki.debian.org/PowerPCSPEPort Patch by Roland Stigge
> 
> Modified:
>    cfe/trunk/lib/Driver/ToolChains.cpp
> 
> Modified: cfe/trunk/lib/Driver/ToolChains.cpp
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Driver/ToolChains.cpp?rev=177161&r1=177160&r2=177161&view=diff
> ==============================================================================
> --- cfe/trunk/lib/Driver/ToolChains.cpp (original)
> +++ cfe/trunk/lib/Driver/ToolChains.cpp Fri Mar 15 11:22:43 2013
> @@ -1149,6 +1149,7 @@ Generic_GCC::GCCInstallationDetector::GC
>   static const char *const PPCTriples[] = {
>     "powerpc-linux-gnu",
>     "powerpc-unknown-linux-gnu",
> +    "powerpc-linux-gnuspe",
>     "powerpc-suse-linux",
>     "powerpc-montavista-linuxspe"
>   };
> @@ -2255,6 +2256,8 @@ static std::string getMultiarchTriple(co
>       return "mipsel-linux-gnu";
>     return TargetTriple.str();
>   case llvm::Triple::ppc:
> +    if (llvm::sys::fs::exists(SysRoot + "/lib/powerpc-linux-gnuspe"))
> +      return "powerpc-linux-gnuspe";
>     if (llvm::sys::fs::exists(SysRoot + "/lib/powerpc-linux-gnu"))
>       return "powerpc-linux-gnu";
>     return TargetTriple.str();
> 
> 
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits