From 03adb248d621063cbbb0ce583cbd2e016ada7a6f Mon Sep 17 00:00:00 2001 From: Tom Tromey Date: Thu, 1 Jul 2021 11:36:58 -0600 Subject: Defer Ada character literal resolution In Ada, an enumeration type can use a character literal as one of the enumerators. The Ada expression parser handles the appropriate conversion. It turns out, though, that this conversion was handled incorrectly. For an expression like TYPE'(EXP), the conversion would be done for any such literal appearing in EXP -- but only the outermost such expression should really be affected. This patch defers the conversion until the resolution phase, fixing the bug. --- gdb/testsuite/gdb.ada/char_enum_overload/foo.adb | 22 +++++++++++++++++ gdb/testsuite/gdb.ada/char_enum_overload/pck.adb | 31 ++++++++++++++++++++++++ gdb/testsuite/gdb.ada/char_enum_overload/pck.ads | 25 +++++++++++++++++++ 3 files changed, 78 insertions(+) create mode 100644 gdb/testsuite/gdb.ada/char_enum_overload/foo.adb create mode 100644 gdb/testsuite/gdb.ada/char_enum_overload/pck.adb create mode 100644 gdb/testsuite/gdb.ada/char_enum_overload/pck.ads (limited to 'gdb/testsuite/gdb.ada/char_enum_overload') diff --git a/gdb/testsuite/gdb.ada/char_enum_overload/foo.adb b/gdb/testsuite/gdb.ada/char_enum_overload/foo.adb new file mode 100644 index 0000000..ee0a0aa --- /dev/null +++ b/gdb/testsuite/gdb.ada/char_enum_overload/foo.adb @@ -0,0 +1,22 @@ +-- Copyright 2021 Free Software Foundation, Inc. +-- +-- This program is free software; you can redistribute it and/or modify +-- it under the terms of the GNU General Public License as published by +-- the Free Software Foundation; either version 3 of the License, or +-- (at your option) any later version. +-- +-- This program is distributed in the hope that it will be useful, +-- but WITHOUT ANY WARRANTY; without even the implied warranty of +-- MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +-- GNU General Public License for more details. +-- +-- You should have received a copy of the GNU General Public License +-- along with this program. If not, see . + +with Pck; use Pck; + +procedure Foo is + Gchar : Global_Enum_Type := Global_Enum_Type'(Overloaded('+')); +begin + Do_Nothing (Gchar'Address); -- STOP +end Foo; diff --git a/gdb/testsuite/gdb.ada/char_enum_overload/pck.adb b/gdb/testsuite/gdb.ada/char_enum_overload/pck.adb new file mode 100644 index 0000000..6aba048 --- /dev/null +++ b/gdb/testsuite/gdb.ada/char_enum_overload/pck.adb @@ -0,0 +1,31 @@ +-- Copyright 2021 Free Software Foundation, Inc. +-- +-- This program is free software; you can redistribute it and/or modify +-- it under the terms of the GNU General Public License as published by +-- the Free Software Foundation; either version 3 of the License, or +-- (at your option) any later version. +-- +-- This program is distributed in the hope that it will be useful, +-- but WITHOUT ANY WARRANTY; without even the implied warranty of +-- MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +-- GNU General Public License for more details. +-- +-- You should have received a copy of the GNU General Public License +-- along with this program. If not, see . + +package body Pck is + procedure Overloaded (Value : Global_Enum_Type) is + begin + null; + end Overloaded; + + function Overloaded (Value : Character) return Global_Enum_Type is + begin + return 'Y'; + end Overloaded; + + procedure Do_Nothing (A : System.Address) is + begin + null; + end Do_Nothing; +end Pck; diff --git a/gdb/testsuite/gdb.ada/char_enum_overload/pck.ads b/gdb/testsuite/gdb.ada/char_enum_overload/pck.ads new file mode 100644 index 0000000..7dc478f --- /dev/null +++ b/gdb/testsuite/gdb.ada/char_enum_overload/pck.ads @@ -0,0 +1,25 @@ +-- Copyright 2021 Free Software Foundation, Inc. +-- +-- This program is free software; you can redistribute it and/or modify +-- it under the terms of the GNU General Public License as published by +-- the Free Software Foundation; either version 3 of the License, or +-- (at your option) any later version. +-- +-- This program is distributed in the hope that it will be useful, +-- but WITHOUT ANY WARRANTY; without even the implied warranty of +-- MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +-- GNU General Public License for more details. +-- +-- You should have received a copy of the GNU General Public License +-- along with this program. If not, see . + +with System; + +package Pck is + type Global_Enum_Type is ('x', 'Y', '+'); + + procedure Overloaded (Value : Global_Enum_Type); + function Overloaded (Value : Character) return Global_Enum_Type; + + procedure Do_Nothing (A : System.Address); +end Pck; -- cgit v1.1