[all-commits] [llvm/llvm-project] bd8f10: [lldb] correct inconsistent order of messages on p...

[José Lira Junior via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: bd8f1068cad06b0f0342ac7ef351bf01c2e27322
      https://github.com/llvm/llvm-project/commit/bd8f1068cad06b0f0342ac7ef351bf01c2e27322
  Author: José Lira Junior <josejunior at 10xengineers.ai>
  Date:   2023-11-24 (Fri, 24 Nov 2023)

  Changed paths:
    M lldb/source/Commands/CommandObjectProcess.cpp
    M lldb/test/Shell/Breakpoint/jit-loader_jitlink_elf.test
    M lldb/test/Shell/Breakpoint/jit-loader_rtdyld_elf.test

  Log Message:
  -----------
  [lldb] correct inconsistent order of messages on process launch (#73173)

Fixes [#68035](https://github.com/llvm/llvm-project/issues/68035), where
an inconsistency in the order of "Process launched" and "Process
stopped" messages occurs during `process launch`.

The fix involves adjusting the message output sequence in
`CommandObjectProcessLaunch::DoExecute` within
`source/Commands/CommandObjectProcess.cpp`. This ensures "Process
launched" consistently precedes "Process stopped" when executing
commands with the '-o' flag, i.e., non-interactive mode.

Upon implementing this change, two tests failed:
`lldb/test/Shell/Breakpoint/jit-loader_jitlink_elf.test` and
`lldb/test/Shell/Breakpoint/jit-loader_rtdyld_elf.test`. These failures
were expected as they relied on the previous, now-corrected message
order. Updating these tests to align with the new message sequence is
part of this PR's scope.