[all-commits] [llvm/llvm-project] e5ea8f: Allow 128-bit discriminants in DWARF variants (#12...

beetrees via All-commits all-commits at lists.llvm.org
Fri Feb 7 18:29:29 PST 2025


  Branch: refs/heads/release/20.x
  Home:   https://github.com/llvm/llvm-project
  Commit: e5ea8f0a4855acd532db6beab8944a6226287a5f
      https://github.com/llvm/llvm-project/commit/e5ea8f0a4855acd532db6beab8944a6226287a5f
  Author: Tom Tromey <tom at tromey.com>
  Date:   2025-02-07 (Fri, 07 Feb 2025)

  Changed paths:
    M llvm/lib/CodeGen/AsmPrinter/DwarfUnit.cpp
    M llvm/lib/CodeGen/AsmPrinter/DwarfUnit.h
    M llvm/test/DebugInfo/Generic/discriminated-union.ll

  Log Message:
  -----------
  Allow 128-bit discriminants in DWARF variants (#125578)

If a variant part has a 128-bit discriminator, then
DwarfUnit::constructTypeDIE will assert.  This patch fixes the problem
by allowing any size of integer to be used here.  This is largely
accomplished by moving part of DwarfUnit::addConstantValue to a new
method.

Fixes #119655

(cherry picked from commit 3c2807624d2006fa8aacf9c6441c9a3034a52b44)


  Commit: 5e36383513a57704b513732b4b152b67933c37d9
      https://github.com/llvm/llvm-project/commit/5e36383513a57704b513732b4b152b67933c37d9
  Author: beetrees <b at beetr.ee>
  Date:   2025-02-07 (Fri, 07 Feb 2025)

  Changed paths:
    M llvm/test/DebugInfo/Generic/discriminated-union.ll

  Log Message:
  -----------
  Fix `llvm/test/DebugInfo/Generic/discriminated-union.ll` on big-endian targets (#125849)

Fixes the failure of the [Solaris/sparcv9
buildbot](https://lab.llvm.org/buildbot/#/builders/13/builds/5103)
caused by #125578.

cc @rorth @tromey @dwblaikie

(cherry picked from commit 34929853bc39d28943373b8a96371a7e81e98917)


Compare: https://github.com/llvm/llvm-project/compare/f92369f2591f...5e36383513a5

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


More information about the All-commits mailing list