[all-commits] [llvm/llvm-project] 5b4100: [lldb-dap] Improve `stackTrace` and `exceptionInfo...

[John Harrison via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 5b4100cc354148a1140546e7f5ac2bf380bc5eff
      https://github.com/llvm/llvm-project/commit/5b4100cc354148a1140546e7f5ac2bf380bc5eff
  Author: John Harrison <harjohn at google.com>
  Date:   2024-09-10 (Tue, 10 Sep 2024)

  Changed paths:
    M lldb/packages/Python/lldbsuite/test/lldbplatformutil.py
    M lldb/packages/Python/lldbsuite/test/tools/lldb-dap/dap_server.py
    M lldb/packages/Python/lldbsuite/test/tools/lldb-dap/lldbdap_testcase.py
    M lldb/test/API/tools/lldb-dap/exception/Makefile
    M lldb/test/API/tools/lldb-dap/exception/TestDAP_exception.py
    A lldb/test/API/tools/lldb-dap/exception/cpp/Makefile
    A lldb/test/API/tools/lldb-dap/exception/cpp/TestDAP_exception_cpp.py
    A lldb/test/API/tools/lldb-dap/exception/cpp/main.cpp
    A lldb/test/API/tools/lldb-dap/exception/main.c
    R lldb/test/API/tools/lldb-dap/exception/main.cpp
    A lldb/test/API/tools/lldb-dap/exception/objc/Makefile
    A lldb/test/API/tools/lldb-dap/exception/objc/TestDAP_exception_objc.py
    A lldb/test/API/tools/lldb-dap/exception/objc/main.m
    A lldb/test/API/tools/lldb-dap/extendedStackTrace/Makefile
    A lldb/test/API/tools/lldb-dap/extendedStackTrace/TestDAP_extendedStackTrace.py
    A lldb/test/API/tools/lldb-dap/extendedStackTrace/main.m
    M lldb/test/API/tools/lldb-dap/stackTrace/TestDAP_stackTrace.py
    M lldb/test/API/tools/lldb-dap/stackTrace/main.c
    M lldb/test/API/tools/lldb-dap/stackTraceMissingFunctionName/TestDAP_stackTraceMissingFunctionName.py
    M lldb/tools/lldb-dap/JSONUtils.cpp
    M lldb/tools/lldb-dap/JSONUtils.h
    M lldb/tools/lldb-dap/README.md
    M lldb/tools/lldb-dap/lldb-dap.cpp
    M lldb/tools/lldb-dap/package.json

  Log Message:
  -----------
  [lldb-dap] Improve `stackTrace` and `exceptionInfo` DAP request handlers (#105905)

Refactoring `stackTrace` to perform frame look ups in a more on-demand
fashion to improve overall performance.

Additionally adding additional information to the `exceptionInfo`
request to report exception stacks there instead of merging the
exception stack into the stack trace. The `exceptionInfo` request is
only called if a stop event occurs with `reason='exception'`, which
should mitigate the performance of `SBThread::GetCurrentException`
calls.

Adding unit tests for exception handling and stack trace supporting.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications