[Lldb-commits] [lldb] [lldb][test] Set target OS for API tests in case of remote testing (PR #96654)

Pavel Labath via lldb-commits lldb-commits at lists.llvm.org
Wed Jun 26 01:47:47 PDT 2024


================
@@ -97,6 +101,9 @@ def finalize_build_dictionary(dictionary):
             dictionary = {}
         dictionary["OS"] = "Android"
         dictionary["PIE"] = 1
+    elif target_is_remote_linux():
----------------
labath wrote:

For consistency, I think we should do this for all linux systems, not just remote ones. (Ideally, I would do it for all systems overall, but I can understand if you're reluctant to do that.)

You can use `getPlatform` to get the platform/os name for both local and remote runs.

https://github.com/llvm/llvm-project/pull/96654


More information about the lldb-commits mailing list