jasonmolenda wrote:

> FYI, I also ran into failures due to this change, when debugging on Windows 
> (with DWARF). See the run in 
> https://github.com/mstorsjo/actions-test/actions/runs/10020326811 vs 
> https://github.com/mstorsjo/actions-test/actions/runs/10020393197. Thanks for 
> reverting; hopefully it’s the same root cause as on other platforms.

oooh that 10020393197 is surprising.  It looks like we hit a breakpoint, did 
`finish` up until a recursive function and it resumed execution without 
stopping.  I haven't seen that one before, and not sure how I could have 
introduced it.  I'll def need to look at my ProcessWindows changes and again 
see if I can't figure out what might be going on.  Aleksandr ran a slightly 
earlier version of the windows changes against the lldb testsuite and didn't 
uncover any problems, but it didn't include this test file.

@mstorsjo is this test source file on-line somewhere, or can you paste it into 
a comment?  I'll try to repo on linux or macos too, maybe it is a unique corner 
case unrelated to Windows that is a problem.  (unlikely, but you never know if 
we might get lucky).  Thanks for the heads-up.

https://github.com/llvm/llvm-project/pull/96260
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to