[all-commits] [llvm/llvm-project] b8dcd0: [NFC][lldb][trace] Rename trace session to trace b...

walter erquinigo via All-commits all-commits at lists.llvm.org
Fri Jun 24 08:41:48 PDT 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: b8dcd0ba26a90166cc76e6d1db23b88656610b2e
      https://github.com/llvm/llvm-project/commit/b8dcd0ba26a90166cc76e6d1db23b88656610b2e
  Author: Walter Erquinigo <wallace at fb.com>
  Date:   2022-06-24 (Fri, 24 Jun 2022)

  Changed paths:
    M lldb/include/lldb/Core/PluginManager.h
    M lldb/include/lldb/Target/Trace.h
    M lldb/include/lldb/lldb-private-interfaces.h
    M lldb/source/Commands/CommandObjectTrace.cpp
    M lldb/source/Core/PluginManager.cpp
    M lldb/source/Plugins/Trace/common/ThreadPostMortemTrace.h
    M lldb/source/Plugins/Trace/intel-pt/CMakeLists.txt
    M lldb/source/Plugins/Trace/intel-pt/TraceCursorIntelPT.h
    M lldb/source/Plugins/Trace/intel-pt/TraceIntelPT.cpp
    M lldb/source/Plugins/Trace/intel-pt/TraceIntelPT.h
    A lldb/source/Plugins/Trace/intel-pt/TraceIntelPTBundleLoader.cpp
    A lldb/source/Plugins/Trace/intel-pt/TraceIntelPTBundleLoader.h
    A lldb/source/Plugins/Trace/intel-pt/TraceIntelPTBundleSaver.cpp
    A lldb/source/Plugins/Trace/intel-pt/TraceIntelPTBundleSaver.h
    M lldb/source/Plugins/Trace/intel-pt/TraceIntelPTJSONStructs.cpp
    M lldb/source/Plugins/Trace/intel-pt/TraceIntelPTJSONStructs.h
    R lldb/source/Plugins/Trace/intel-pt/TraceIntelPTSessionFileParser.cpp
    R lldb/source/Plugins/Trace/intel-pt/TraceIntelPTSessionFileParser.h
    R lldb/source/Plugins/Trace/intel-pt/TraceIntelPTSessionSaver.cpp
    R lldb/source/Plugins/Trace/intel-pt/TraceIntelPTSessionSaver.h
    M lldb/source/Target/Trace.cpp
    M lldb/test/API/commands/trace/TestTraceLoad.py

  Log Message:
  -----------
  [NFC][lldb][trace] Rename trace session to trace bundle

As previously discussed with @jj10306, we didn't really have a name for
the post-mortem (or offline) trace session representation, which is in
fact a folder with a bunch of files. We decided to call this folder
"trace bundle", and the main JSON file in it "trace bundle description
file". This naming is pretty decent, so I'm refactoring all the existing
code to account for that.

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




More information about the All-commits mailing list