[all-commits] [llvm/llvm-project] a1cfa7: Fix SectionList::ReplaceSection to not replace inc...

[Greg Clayton via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: a1cfa786c96fee1ce63d034674d7aeb4cc9cc5d9
      https://github.com/llvm/llvm-project/commit/a1cfa786c96fee1ce63d034674d7aeb4cc9cc5d9
  Author: Greg Clayton <gclayton at fb.com>
  Date:   2026-06-22 (Mon, 22 Jun 2026)

  Changed paths:
    M lldb/include/lldb/Core/Section.h
    M lldb/source/Core/Section.cpp
    M lldb/source/Plugins/ObjectFile/ELF/ObjectFileELF.cpp
    M lldb/source/Plugins/SymbolVendor/ELF/SymbolVendorELF.cpp
    M lldb/source/Plugins/SymbolVendor/PECOFF/SymbolVendorPECOFF.cpp
    M lldb/source/Plugins/SymbolVendor/wasm/SymbolVendorWasm.cpp
    A lldb/test/Shell/ObjectFile/ELF/build-id-case-debug-only.yaml

  Log Message:
  -----------
  Fix SectionList::ReplaceSection to not replace incorrect section. (#204677)

We use SectionList::ReplaceSection to check for some sections in the
main object file and in separate debug info files. It was relying on
section IDs being consistent between different individual section lists
in different object files which does not work. I fixed this by not using
a section ID when replacing a section, but using the section shared
pointer so there can be no errors.



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