[all-commits] [llvm/llvm-project] 9d56d9: [llvm-debuginfo-analyzer] Fix memory leak reported...

Carlos Alberto Enciso via All-commits all-commits at lists.llvm.org
Tue Nov 1 23:08:32 PDT 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 9d56d9a95fcf065f9542e49300e0d6af66883e73
      https://github.com/llvm/llvm-project/commit/9d56d9a95fcf065f9542e49300e0d6af66883e73
  Author: Carlos Alberto Enciso <carlos.alberto.enciso at gmail.com>
  Date:   2022-11-02 (Wed, 02 Nov 2022)

  Changed paths:
    M llvm/lib/DebugInfo/LogicalView/Readers/LVBinaryReader.cpp
    M llvm/lib/DebugInfo/LogicalView/Readers/LVELFReader.cpp
    M llvm/test/tools/llvm-debuginfo-analyzer/DWARF/pr-incorrect-logical-instructions.test
    M llvm/unittests/DebugInfo/LogicalView/ELFReaderTest.cpp

  Log Message:
  -----------
  [llvm-debuginfo-analyzer] Fix memory leak reported by sanitizers.

The sanitizer reported memory leak issues; the command line
used in the test case is:

llvm-debuginfo-analyzer --attribute=level
                        --print=instructions
                        pr-incorrect-instructions-dwarf-clang.o

When dealing with logical instruction lines associated with
an artificial logical scope, skip the process of finding
their enclosing scope. Just add them to the scope.

Create logical debug lines only if the command line specifies:
  --print=lines or --print=elements or --print=all

Reviewed By: jryans, vitalybuka

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




More information about the All-commits mailing list