[cfe-commits] [PATCH] Android support in Clang driver

[Evgeniy Stepanov]

Hi,

thanks for looking at the patch!
I've merged addAsanRT* into one function and wrote some tests. Please
take another look.

On Tue, Apr 24, 2012 at 6:50 PM, James Molloy <james.molloy at arm.com> wrote:
> Hi Evgeniy,
>
> You're right, it is ugly, but that unfortunately is par for the course for a driver patch :(
>
> You change several functions to take a Triple argument and switch on that inside, but for addAsanRT you add a brand new function. It'd be cleaner to do the same and add a Triple argument, modifying the behaviour of addAsanRT depending on Linux or Android.
>
> Also, testcases...!
>
> Cheers,
>
> James
>
> -----Original Message-----
> From: cfe-commits-bounces at cs.uiuc.edu [mailto:cfe-commits-bounces at cs.uiuc.edu] On Behalf Of Evgeniy Stepanov
> Sent: 24 April 2012 15:15
> To: cfe-commits at cs.uiuc.edu
> Subject: [cfe-commits] [PATCH] Android support in Clang driver
>
> Hi,
>
> this patch add Android support to linuxtools::Link. This allows using
> Clang as a drop-in GCC replacement in Android NDK.
>
> It is kind of ugly, sorry about that. There is simply too many small
> differencies in how things are done between android and linux.
>
>
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: android-driver.patch
Type: application/octet-stream
Size: 11052 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20120425/898805a1/attachment.obj>