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/test/API/functionalities/thread/concurrent_events/TestConcurrentBatchedBreakpointStepOver.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
--- TestConcurrentBatchedBreakpointStepOver.py 2026-02-07 06:43:38.000000 +0000
+++ TestConcurrentBatchedBreakpointStepOver.py 2026-02-07 06:46:24.646203 +0000
@@ -13,11 +13,10 @@
from lldbsuite.test.lldbtest import TestBase
@skipIfWindows
class ConcurrentBatchedBreakpointStepOver(ConcurrentEventsBase):
-
@skipIf(triple="^mips")
@expectedFailureAll(
archs=["aarch64"], oslist=["freebsd"], bugnumber="llvm.org/pr49433"
)
def test(self):
@@ -28,13 +27,11 @@
# Enable logging to capture optimization messages and GDB packets.
# Both categories must be in a single command because successive
# 'log enable lldb <cat> -f <file>' calls redirect the entire 'lldb'
# channel to the new file, leaving the old file empty.
lldb_logfile = self.getBuildArtifact("lldb-log.txt")
- self.runCmd(
- "log enable lldb step break -f {}".format(lldb_logfile)
- )
+ self.runCmd("log enable lldb step break -f {}".format(lldb_logfile))
gdb_logfile = self.getBuildArtifact("gdb-remote-log.txt")
self.runCmd("log enable gdb-remote packets -f {}".format(gdb_logfile))
# Run with 10 breakpoint threads - the pseudo_barrier in main.cpp
@@ -81,13 +78,11 @@
with open(gdb_logfile, "r") as f:
for line in f:
if "send packet: $" not in line:
continue
# Match z0,<addr> (disable) or Z0,<addr> (enable)
- m = re.search(
- r'send packet: \$([Zz])0,([0-9a-fA-F]+),', line
- )
+ m = re.search(r"send packet: \$([Zz])0,([0-9a-fA-F]+),",
line)
if m and m.group(2) == bp_addr_hex:
if m.group(1) == "Z":
if not initial_Z0_seen:
# Skip the initial breakpoint set
initial_Z0_seen = True
@@ -96,12 +91,11 @@
else:
z0_count += 1
else:
self.fail("gdb-remote log file not found")
- print("\n--- Breakpoint packet summary for {} ---".format(
- thread_bp_addr))
+ print("\n--- Breakpoint packet summary for {}
---".format(thread_bp_addr))
print(" z0 (disable) packets: {}".format(z0_count))
print(" Z0 (re-enable) packets: {}".format(Z0_count))
# With the optimization: 1 z0 (disable once) + 1 Z0 (re-enable once)
# Without optimization: N z0 + N Z0 (one pair per thread)
``````````
</details>
https://github.com/llvm/llvm-project/pull/180101
_______________________________________________
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
