[Lldb-commits] [PATCH] D93874: [process] fix exec support on Linux

walter erquinigo via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Mon Jan 4 14:40:09 PST 2021


wallace added inline comments.


================
Comment at: lldb/test/API/functionalities/exec/TestExec.py:96
         # Run and we should stop due to exec
         process.Continue()
 
----------------
clayborg wrote:
> Do we need this continue if we did the step above? How does this test still work if the step was supposed to step over the exec?
It seems to me that the StepInstruction before happens before the exec is reached, and after this process.Continue() exec will be reached


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D93874/new/

https://reviews.llvm.org/D93874



More information about the lldb-commits mailing list