================
@@ -802,9 +802,14 @@ def canRunLibcxxTests():
platform = lldbplatformutil.getPlatform()
- if lldbplatformutil.target_is_android() or
lldbplatformutil.platformIsDarwin():
+ if lldbplatformutil.target_is_android():
return True, "libc++ always present"
+ if lldbplatformutil.platformIsDarwin():
+ if not configuration.libcxx_include_dir or not
configuration.libcxx_library_dir:
+ return False, "API tests require a locally built libc++."
----------------
JDevlieghere wrote:
This seems too generic?
```suggestion
return False, "libc++ tests require a locally built libc++"
```
https://github.com/llvm/llvm-project/pull/199262
_______________________________________________
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits