[Lldb-commits] [PATCH] Add a couple more paths where to look for just-build Clang

Kuba Brecka kuba.brecka at gmail.com
Mon Oct 13 10:27:33 PDT 2014


You're right, that's probably a better approach. However, on OS X, it's quite possible your default compiler will not understand -fsanitize=address, and running just "./dotest.py" will have the ASan tests fail. I think we should have a @requiresAsanCompiler annotation that would detect and skip the test, if your compiler (either the default one or the one you set via LLDB_TEST_COMPILER) doesn't have ASan. 

Kuba

Sent from my iPhone

> On Oct 13, 2014, at 9:52 AM, Zachary Turner <zturner at google.com> wrote:
> 
> Are you using the CMake build?  Even this will fail if you have multiple clangs and you want it to choose a specific one.  I would suggest setting the LLDB_TEST_COMPILER CMake variable.
> 
>> On Fri, Oct 10, 2014 at 8:19 PM, Kuba Brecka <kuba.brecka at gmail.com> wrote:
>> Some LLDB build scripts seem to put the built Clang into different paths, let's add a couple more paths where to look for Clang. Currently used only by LLDB's ASan tests.
>> 
>> http://reviews.llvm.org/D5740
>> 
>> Files:
>>   test/lldbtest.py
>> 
>> _______________________________________________
>> lldb-commits mailing list
>> lldb-commits at cs.uiuc.edu
>> http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits
> 
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20141013/200dafe8/attachment.html>


More information about the lldb-commits mailing list