[llvm-commits] [llvm] r123990 - in /llvm/trunk: include/llvm/ADT/Triple.h lib/Support/Triple.cpp unittests/ADT/TripleTest.cpp

[İsmail Dönmez]

Hi;

On Mon, Jan 24, 2011 at 1:30 PM, Renato Golin <Renato.Golin at arm.com> wrote:

> On 01/24/11 10:12, Duncan Sands wrote:
> > I don't see the point of having "none" be turned into "NoVendor" rather
> than
> > "UnknownVendor" if you aren't go to do anything with that information
> anyway.
> > However if you have a good reason to want "none" to be parsed then I will
> > drop my objections to "none" :)
>
> I guess that falls together with the LLVM/Clang triple implementation.
>
> Clang should interpret UnknownVendor to "none" and not LLVM changed to
> have a dubious and duplicated "none".
>
>
> > In short, whether "none" is parsed at all, or only parsed as a
> > valid "O/S", or parsed as a valid vendor and a valid O/S etc depends on
> what
> > triples occur in the real world and what you plan to do with the
> information.
>
> I agree. My mistake was to try and not break anything that was already
> working.
>
> Some ARM triples I found in the wild...
>
>  -- From CodeSourcery:
>
> * arm-none-eabi
>   none I "guess" it's for vendor, and no OS
>
> * arm-none-linux-gnueabi
>   again, it's vendor
>

Android started using arm-linux-androideabi recently, old one was arm-eabi.

Regards,
ismail
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20110124/f861942b/attachment.html>