[all-commits] [llvm/llvm-project] 408032: DwarfDebug: Minor condition cleanups

[David Blaikie via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 40803282b788e1c6f8e035c5359420954b275355
      https://github.com/llvm/llvm-project/commit/40803282b788e1c6f8e035c5359420954b275355
  Author: David Blaikie <dblaikie at gmail.com>
  Date:   2023-03-29 (Wed, 29 Mar 2023)

  Changed paths:
    M llvm/lib/CodeGen/AsmPrinter/DwarfDebug.cpp

  Log Message:
  -----------
  DwarfDebug: Minor condition cleanups

Sink/share a few common conditions.


  Commit: a73d354024f668b30dc828366a40c50dab8b4159
      https://github.com/llvm/llvm-project/commit/a73d354024f668b30dc828366a40c50dab8b4159
  Author: David Blaikie <dblaikie at gmail.com>
  Date:   2023-03-29 (Wed, 29 Mar 2023)

  Changed paths:
    M llvm/lib/CodeGen/AsmPrinter/DwarfCompileUnit.cpp
    M llvm/lib/CodeGen/AsmPrinter/DwarfDebug.cpp
    M llvm/lib/CodeGen/AsmPrinter/DwarfDebug.h
    M llvm/test/DebugInfo/X86/debug_addr.ll
    A llvm/test/DebugInfo/X86/ranges_always_default.ll

  Log Message:
  -----------
  DWARF: Enable "ranges always" under Split DWARF by default

Given the intent of Split DWARF is to minimize .o file size it seems
like adequate signal that it's worth a minor tradeoff in .dwo size to
significantly reduce .o size (though it doesn't reduce linked executable
size - the cost is mostly in the static relocations resolved by the
linker).


Compare: https://github.com/llvm/llvm-project/compare/b02db1dc78ee...a73d354024f6