================
@@ -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

Reply via email to