[PATCH] [ARM] Remove dead code for target architectures with a '-' in the name

Bob Wilson bob.wilson at apple.com
Tue Sep 17 17:56:31 PDT 2013


I looked at this very briefly.  I don't think the hyphenated architecture names get used much, but we shouldn't break them without a compelling reason.  Brad, your patch seems to be based on the assumption that the architecture name will be coming from the triple.  That is one possibility, but it can also come from the -march option.  You are right that the hyphenated names won't work in triples, but they work fine with -march.

On Sep 17, 2013, at 4:41 AM, Renato Golin <renato.golin at linaro.org> wrote:

> 
>  Well, I'm fine in removing them all, but other folks might have some reservation. Check with Jim Grosbach or Bob Wilson before committing.
> 
> http://llvm-reviews.chandlerc.com/D1695
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits




More information about the cfe-commits mailing list