hfinkel added a comment.

In https://reviews.llvm.org/D25686#571894, @mgorny wrote:

> In https://reviews.llvm.org/D25686#571857, @hfinkel wrote:
>
> > It seems like we should teach Triple how to parse this triples correctly 
> > (i.e. to recognize that the vendor is ARM), and then canonicalize the 
> > environment to GNUEABIHF (or whatever)?
>
>
> I've attempted that but altering the triple resulted in clang being unable to 
> find gcc files (since the triple didn't match anymore). My previous patch is 
> available here: https://595834.bugs.gentoo.org/attachment.cgi?id=450004. If 
> you think that I've did something stupid-wrong there and there's a better way 
> of doing this via Triple API, I'll gladly update the patch.


I think that you should update the triple class to return meaningful results 
and then also enhance Clang's search logic to find the GCC installation 
directory.


https://reviews.llvm.org/D25686



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to