[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