On 9 Feb 2018, at 22:30, Dimitry Andric <dimi...@andric.com> wrote:
> 
> On 9 Feb 2018, at 22:11, Dimitry Andric via Openmp-dev 
> <openmp-...@lists.llvm.org> wrote:
>> 
>> On 9 Feb 2018, at 20:40, Dimitry Andric via cfe-dev <cfe-...@lists.llvm.org> 
>> wrote:
>>> 
>>>> On 9 Feb 2018, at 10:20, Hans Wennborg <h...@chromium.org> wrote:
>> ...
>>>> What are all these test failures? Does it seems like they have a
>>>> common root cause and do we have a bug for it?
>> ...
>>> The Clang Tools and Extra Tools Unit tests all appear to crash with:
>>> 
>>>  exception_ptr not yet implemented
>> 
>> This turns out to be caused by libc++ being compiled without -DLIBCXXRT.  
>> (In the FreeBSD base system build, we always add this option, so libc++ 
>> knows how to handle exceptions.)
>> 
>> In the libc++ CMakeFiles, it appears to be governed by 
>> LIBCXX_CXX_ABI_LIBNAME, but it isn't being set to the correct value of 
>> "cxxrt" on FreeBSD.  I am going to try the following diff:
>> 
>> --- llvm.src/projects/libcxx/CMakeLists.txt
>> +++ llvm.src/projects/libcxx/CMakeLists.txt
>> @@ -135,6 +135,8 @@
>>  elseif (APPLE)
>>    set(LIBCXX_CXX_ABI_LIBNAME "libcxxabi")
>>    set(LIBCXX_CXX_ABI_SYSTEM 1)
>> +  elseif (CMAKE_SYSTEM_NAME MATCHES "FreeBSD")
>> +    set(LIBCXX_CXX_ABI_LIBNAME "libcxxrt")
>>  else()
>>    set(LIBCXX_CXX_ABI_LIBNAME "default")
>>  endif()
> 
> ... and unfortunately that didn't work, since the CMakeFiles are unable to 
> find the libcxxrt headers:
> 
> CMake Warning at projects/libcxx/cmake/Modules/HandleLibCXXABI.cmake:67 
> (message):
>  Failed to find cxxabi.h
> Call Stack (most recent call first):
>  projects/libcxx/cmake/Modules/HandleLibCXXABI.cmake:112 (setup_abi_lib)
>  projects/libcxx/CMakeLists.txt:428 (include)

Ok, this turned out to be easier than I thought.  After applying 
https://reviews.llvm.org/D43166, the number of failed tests drops roughly by 
half (from 896 to 512):

  Expected Passes    : 45381
  Expected Failures  : 185
  Unsupported Tests  : 2937
  Unexpected Passes  : 1
  Unexpected Failures: 521

I am going to have a look at some other low hanging fruit, and I have also 
created a few PRs to merge test changes into 6.0.

-Dimitry

Attachment: signature.asc
Description: Message signed with OpenPGP

_______________________________________________
lldb-dev mailing list
lldb-dev@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-dev

Reply via email to