[llvm-commits] [PATCH] Re-introduce llvm::sys::getHostTriple() function.

[Kaylor, Andrew]

Hi Peter,

Sorry to take so long to get back to you about this.

I just tried out your patch and it does not work on Windows.  It seems that 'sed' is required to build 'llvm-config' so 'llvm-config' is not built on Windows.

-Andy

-----Original Message-----
From: Peter Collingbourne [mailto:peter at pcc.me.uk] 
Sent: Tuesday, January 08, 2013 11:58 AM
To: Kaylor, Andrew
Cc: llvm-commits at cs.uiuc.edu; spop at codeaurora.org; joerg at NetBSD.org
Subject: Re: [llvm-commits] [PATCH] Re-introduce llvm::sys::getHostTriple() function.

On Sat, Jan 05, 2013 at 01:16:00AM +0000, Kaylor, Andrew wrote:
> I can test your patch on Windows if you can find a way to get it to be consistent.  I looked into something like that before and the best I could come up with was to disable the MCJIT tests in cases that I expected to have problems.  It seemed that in the build files the host triple and the target triple were specified using different naming schemes and I couldn't figure out a simple way to convert one to the other.  In your case, since you just want to always use the host triple (but possibly with the 32/64-bit conversion) it might be easier.

Can you please test the patch I sent out?  I decided that exposing the triple via llvm-config was probably the best way for lit to access it.

Thanks,
--
Peter