[PATCH] [ASan] Fixed path to Android bin directory

Greg Fitzgerald garious at gmail.com
Fri Oct 11 10:10:20 PDT 2013


Evgeniy, Renato,

Thanks for your replies.  You guys are correct.  My patch is no good.
Sorry for the noise.

-Greg


On Fri, Oct 11, 2013 at 2:09 AM, Evgeniy Stepanov
<eugeni.stepanov at gmail.com> wrote:
> After reading the doc some more, I think it tries to say that -B
> values are translated verbatim to -isystem, -L and also prepended to
> binary paths, which means users must specify multiple -B options for
> all interesting directories.
>
> But in this case we only care about clang, which seems to find
> everything (on my machine) with a single -B option pointing to the
> root of the toolchain. Greg, could you figure out why this does not
> work for you?
>
>
> On Fri, Oct 11, 2013 at 1:06 PM, Renato Golin <renato.golin at linaro.org> wrote:
>> On 11 October 2013 10:02, Renato Golin <renato.golin at linaro.org> wrote:
>>>
>>> which would imply that the "/bin" should be passed. And since
>>> "/usr/lib/gcc" is there the libraries live, not the binaries, I think that
>>> part of the manual is just wrong. Possibly copy&paste error?
>>
>>
>> Ok, I think we both got it wrong, Greg. Evgeniy is right.
>>
>> This is not the path for other toolchain binaries, but
>> binaries/libraries/headers for the compiler itself (like lto-wrapper for
>> gcc).
>>
>> So, what you want is to change the binary discovery mechanism when looking
>> for the android toolchain.
>>
>> cheers,
>> --renato
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits



More information about the llvm-commits mailing list