[all-commits] [llvm/llvm-project] a42675: [lldb] Add newline in output of `target modules lo...

[alvinhochun via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: a426753ef0290362a0268f917dba46d6b9744b02
      https://github.com/llvm/llvm-project/commit/a426753ef0290362a0268f917dba46d6b9744b02
  Author: Alvin Wong <alvin at alvinhc.com>
  Date:   2022-09-27 (Tue, 27 Sep 2022)

  Changed paths:
    M lldb/source/Commands/CommandObjectTarget.cpp
    A lldb/test/Shell/Commands/Inputs/symbols.yaml
    A lldb/test/Shell/Commands/command-target-modules-lookup.test

  Log Message:
  -----------
  [lldb] Add newline in output of `target modules lookup`

This adds a line break between each result address in the output of the
lldb command `target modules lookup`. Before this change, a new address
result will be printed on the same line as the summary of the last
result, making the output difficult to view.

Also adds a test for this command.

Reviewed By: labath

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


  Commit: f4991bfa891eaf9f36aa3b79da6fed4228b69de9
      https://github.com/llvm/llvm-project/commit/f4991bfa891eaf9f36aa3b79da6fed4228b69de9
  Author: Alvin Wong <alvin at alvinhc.com>
  Date:   2022-09-27 (Tue, 27 Sep 2022)

  Changed paths:
    M lldb/source/Commands/CommandObjectTarget.cpp
    M lldb/test/Shell/SymbolFile/absolute-symbol.test

  Log Message:
  -----------
  [lldb] Improve display of absolute symbol lookup

When running `target module lookup` command, show the name of absolute
symbols. Also fix indentation issue after printing an absolute symbol.

Reviewed By: clayborg, DavidSpickett

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


Compare: https://github.com/llvm/llvm-project/compare/8427c836f7b4...f4991bfa891e