aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorGhjuvan Lacambre <lacambre@adacore.com>2021-04-22 17:13:19 +0200
committerPierre-Marie de Rodat <derodat@adacore.com>2021-07-05 13:09:08 +0000
commit767f9cb1f81bdeb55a7b5fbbccb3c057f37e2fae (patch)
treeef3594217d93400b11f1948b7734c54ddb17c480
parentafe5959e11727f6ff276bc1a7c62181306157504 (diff)
downloadgcc-767f9cb1f81bdeb55a7b5fbbccb3c057f37e2fae.zip
gcc-767f9cb1f81bdeb55a7b5fbbccb3c057f37e2fae.tar.gz
gcc-767f9cb1f81bdeb55a7b5fbbccb3c057f37e2fae.tar.bz2
[Ada] Fix overriding subprogram being incorrectly seen as returning
gcc/ada/ * sem_ch3.adb (Check_Abstract_Overriding): Check for renamings.
-rw-r--r--gcc/ada/sem_ch3.adb26
1 files changed, 21 insertions, 5 deletions
diff --git a/gcc/ada/sem_ch3.adb b/gcc/ada/sem_ch3.adb
index 7dfb5c8..7a24298 100644
--- a/gcc/ada/sem_ch3.adb
+++ b/gcc/ada/sem_ch3.adb
@@ -11149,12 +11149,28 @@ package body Sem_Ch3 is
if Present (Overridden_Operation (Subp))
and then No_Return (Overridden_Operation (Subp))
- and then not No_Return (Subp)
then
- Error_Msg_N ("overriding subprogram & must be No_Return", Subp);
- Error_Msg_N
- ("\since overridden subprogram is No_Return (RM 6.5.1(6/2))",
- Subp);
+
+ -- If the subprogram is a renaming, check that the renamed
+ -- subprogram is No_Return.
+
+ if Present (Renamed_Or_Alias (Subp)) then
+ if not No_Return (Renamed_Or_Alias (Subp)) then
+ Error_Msg_N ("subprogram & must be No_Return",
+ Renamed_Or_Alias (Subp));
+ Error_Msg_N ("\since renaming & overrides No_Return "
+ & "subprogram (RM 6.5.1(6/2))",
+ Subp);
+ end if;
+
+ -- Make sure that the subprogram itself is No_Return.
+
+ elsif not No_Return (Subp) then
+ Error_Msg_N ("overriding subprogram & must be No_Return", Subp);
+ Error_Msg_N
+ ("\since overridden subprogram is No_Return (RM 6.5.1(6/2))",
+ Subp);
+ end if;
end if;
-- If the operation is a wrapper for a synchronized primitive, it