github-actions[bot] wrote:
<!--LLVM CODE FORMAT COMMENT: {darker}-->
:warning: Python code formatter, darker found issues in your code. :warning:
<details>
<summary>
You can test this locally with the following command:
</summary>
``````````bash
darker --check --diff -r origin/main...HEAD
lldb/packages/Python/lldbsuite/test/decorators.py
lldb/test/Shell/helper/toolchain.py llvm/utils/lit/lit/llvm/config.py
``````````
:warning:
The reproduction instructions above might return results for more than one PR
in a stack if you are using a stacked PR workflow. You can limit the results by
changing `origin/main` to the base branch/commit you want to compare against.
:warning:
</details>
<details>
<summary>
View the diff from darker here.
</summary>
``````````diff
--- lldb/packages/Python/lldbsuite/test/decorators.py 2025-11-21
19:45:46.000000 +0000
+++ lldb/packages/Python/lldbsuite/test/decorators.py 2025-11-21
22:26:04.891245 +0000
@@ -1056,10 +1056,11 @@
return "Compiler cannot compile with -fsanitize=address"
return None
return skipTestIfFn(is_compiler_with_address_sanitizer)(func)
+
def skipUnlessBoundsSafety(func):
"""Decorate the item to skip test unless Clang -fbounds-safety is
supported."""
def is_compiler_with_bounds_safety():
if not _compiler_supports(lldbplatformutil.getCompiler(),
"-fbounds-safety"):
--- llvm/utils/lit/lit/llvm/config.py 2025-11-21 19:45:46.000000 +0000
+++ llvm/utils/lit/lit/llvm/config.py 2025-11-21 22:26:05.243437 +0000
@@ -292,14 +292,16 @@
return (stdout, stderr)
except OSError:
self.lit_config.fatal("Could not run process %s" % command)
def check_process_success(self, command):
- cp = subprocess.run(command,
- stdout=subprocess.DEVNULL,
- stderr=subprocess.DEVNULL,
- env=self.config.environment)
+ cp = subprocess.run(
+ command,
+ stdout=subprocess.DEVNULL,
+ stderr=subprocess.DEVNULL,
+ env=self.config.environment,
+ )
if cp.returncode == 0:
return True
return False
def feature_config(self, features):
@@ -354,12 +356,12 @@
if additional_flags is None:
additional_flags = []
# Invoke the clang driver to see if it supports the `-fbounds-safety`
# flag. Only the downstream implementation has this flag so this is
# a simple way to check if the full implementation is available or not.
- cmd = [ self.config.clang ] + additional_flags
- cmd += ['-fbounds-safety', '-###']
+ cmd = [self.config.clang] + additional_flags
+ cmd += ["-fbounds-safety", "-###"]
if self.check_process_success(cmd):
return True
return False
# On macOS, LSan is only supported on clang versions 5 and higher
``````````
</details>
https://github.com/llvm/llvm-project/pull/169112
_______________________________________________
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
