[all-commits] [llvm/llvm-project] 230339: Make "process attach -c" work correctly, and add a...

jimingham via All-commits all-commits at lists.llvm.org
Wed Sep 29 19:40:43 PDT 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 2303391d1f543f4e57f9ed0fc68bad2d4cf890dc
      https://github.com/llvm/llvm-project/commit/2303391d1f543f4e57f9ed0fc68bad2d4cf890dc
  Author: Jim Ingham <jingham at apple.com>
  Date:   2021-09-29 (Wed, 29 Sep 2021)

  Changed paths:
    M lldb/source/Commands/CommandObjectProcess.cpp
    M lldb/test/API/commands/process/attach/TestProcessAttach.py

  Log Message:
  -----------
  Make "process attach -c" work correctly, and add a test for it.

The issue here was that we were not updating the interpreter's
execution context when calling HandleCommand to continue the process.
Since we had just created the process, it wasn't in the interpreter's
execution context so HandleCommand failed at CheckRequirements.  The
patch fixes that by passing the process execution context directly
to HandleCommand.

Differential Revision: https://reviews.llvm.org/D110787




More information about the All-commits mailing list