felipepiovezan wrote:

> Sorry to clarify. I think based on @felipepiovezan comment here: [#108870 
> (comment)](https://github.com/llvm/llvm-project/pull/108870#issuecomment-2444481588)
> 
> On Clang 17 we compile fine, just the assertion fails to find the expected 
> function name (because of the `std::__1` inline namespace):
> 
> ```
> FAIL: test_frame_recognizer 
> (TestLibcxxInternalsRecognizer.LibCxxInternalsRecognizerTestCase)
>    Test that implementation details of libc++ are hidden
> ----------------------------------------------------------------------
> Traceback (most recent call last):
>   File 
> "/Users/ec2-user/jenkins/workspace/llvm.org/lldb-cmake-matrix/llvm-project/lldb/test/API/lang/cpp/libcxx-internals-recognizer/TestLibcxxInternalsRecognizer.py",
>  line 60, in test_frame_recognizer
>     self.assertIn(
> AssertionError: 'ranges::__sort::operator()' not found in 
> 'std::__1::__wrap_iter<int*> 
> std::__1::ranges::__sort::__fn::operator()[abi:ue170006]<std::__1::__wrap_iter<int*>,
>  std::__1::__wrap_iter<int*>, bool (*)(int, int), 
> std::__1::identity>(std::__1::__wrap_iter<int*>, std::__1::__wrap_iter<int*>, 
> bool (*)(int, int), std::__1::identity) const'
> Config=x86_64-/Users/ec2-user/jenkins/workspace/llvm.org/lldb-cmake-matrix/clang_1706_build/bin/clang
> ```
> 
> So really we should skip for <= 15. And fix the assertion for 17+

Yeah, I'm just now hitting this assertion on our downstream fork tests too

https://github.com/llvm/llvm-project/pull/114122
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to