diff options
author | Pavel Labath <pavel@labath.sk> | 2022-03-15 16:23:43 +0100 |
---|---|---|
committer | Pavel Labath <pavel@labath.sk> | 2022-03-15 16:23:43 +0100 |
commit | be09f83760ebe7cf698746d5504976ad82679815 (patch) | |
tree | 207733eebbe532c9134b90d0ef5d707aef383560 /lldb/packages/Python/lldbsuite/test/dotest.py | |
parent | 06c5d74090ff00346b8c286bcc27fffbc529cd45 (diff) | |
download | llvm-be09f83760ebe7cf698746d5504976ad82679815.zip llvm-be09f83760ebe7cf698746d5504976ad82679815.tar.gz llvm-be09f83760ebe7cf698746d5504976ad82679815.tar.bz2 |
Revert "[lldb/test] Make category-skipping logic "platform"-independent"
This reverts commit dddf4ce034a8e06cc1351492dceece3fa2344c14. It breaks
a couple of tests on macos.
Diffstat (limited to 'lldb/packages/Python/lldbsuite/test/dotest.py')
-rw-r--r-- | lldb/packages/Python/lldbsuite/test/dotest.py | 26 |
1 files changed, 12 insertions, 14 deletions
diff --git a/lldb/packages/Python/lldbsuite/test/dotest.py b/lldb/packages/Python/lldbsuite/test/dotest.py index 2d8bf53..ce01146 100644 --- a/lldb/packages/Python/lldbsuite/test/dotest.py +++ b/lldb/packages/Python/lldbsuite/test/dotest.py @@ -822,9 +822,9 @@ def checkObjcSupport(): configuration.skip_categories.append("objc") def checkDebugInfoSupport(): - from lldbsuite.test import lldbplatformutil + import lldb - platform = lldbplatformutil.getPlatform() + platform = lldb.selected_platform.GetTriple().split('-')[2] compiler = configuration.compiler for cat in test_categories.debug_info_categories: if cat in configuration.categories_list: @@ -840,14 +840,14 @@ def checkDebugServerSupport(): skip_msg = "Skipping %s tests, as they are not compatible with remote testing on this platform" if lldbplatformutil.platformIsDarwin(): configuration.skip_categories.append("llgs") - if configuration.lldb_platform_name: + if lldb.remote_platform: # <rdar://problem/34539270> configuration.skip_categories.append("debugserver") if configuration.verbose: print(skip_msg%"debugserver"); else: configuration.skip_categories.append("debugserver") - if configuration.lldb_platform_name and lldbplatformutil.getPlatform() == "windows": + if lldb.remote_platform and lldbplatformutil.getPlatform() == "windows": configuration.skip_categories.append("llgs") if configuration.verbose: print(skip_msg%"lldb-server"); @@ -881,16 +881,7 @@ def run_suite(): import lldb lldb.SBDebugger.Initialize() - checkLibcxxSupport() - checkLibstdcxxSupport() - checkWatchpointSupport() - checkDebugInfoSupport() - checkDebugServerSupport() - checkObjcSupport() - checkForkVForkSupport() - # Use host platform by default. - lldb.remote_platform = None lldb.selected_platform = lldb.SBPlatform.GetHostPlatform() # Now we can also import lldbutil @@ -901,7 +892,6 @@ def run_suite(): (configuration.lldb_platform_name)) lldb.remote_platform = lldb.SBPlatform( configuration.lldb_platform_name) - lldb.selected_platform = lldb.remote_platform if not lldb.remote_platform.IsValid(): print( "error: unable to create the LLDB platform named '%s'." % @@ -948,6 +938,14 @@ def run_suite(): # Note that it's not dotest's job to clean this directory. lldbutil.mkdir_p(configuration.test_build_dir) + checkLibcxxSupport() + checkLibstdcxxSupport() + checkWatchpointSupport() + checkDebugInfoSupport() + checkDebugServerSupport() + checkObjcSupport() + checkForkVForkSupport() + print("Skipping the following test categories: {}".format(configuration.skip_categories)) for testdir in configuration.testdirs: |