[all-commits] [llvm/llvm-project] 51fac7: [flang][debug] Don't confuse count with upperBound...

[Abid Qadeer via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 51fac7747f8fb5bfa5e32f5d05e9d807c544a650
      https://github.com/llvm/llvm-project/commit/51fac7747f8fb5bfa5e32f5d05e9d807c544a650
  Author: Abid Qadeer <haqadeer at amd.com>
  Date:   2024-07-10 (Wed, 10 Jul 2024)

  Changed paths:
    M flang/lib/Optimizer/Transforms/DebugTypeGenerator.cpp
    M flang/test/Integration/debug-assumed-shape-array.f90
    M flang/test/Transforms/debug-assumed-shape-array.fir
    M flang/test/Transforms/debug-ptr-type.fir

  Log Message:
  -----------
  [flang][debug] Don't confuse count with upperBound. (#98174)

The code that handled allocatable array had swapped `count` with
`upperBound`. This did not get caught earlier as all the example were
using 1 as `lowerBound`.

Fixes #98166.

With the fix in place, the GDB now correctly handles the case pointed in
the bug ticket.
(gdb) p min::alloc2d
$2 = ((0, 0, 0) (0, 0, 0) (0, 0, 0) (0, 0, 0) (0, 0, 0))
(gdb) ptype min::alloc2d
type = integer, allocatable (-1:1,-2:2)



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