diff options
author | Ghjuvan Lacambre <lacambre@adacore.com> | 2021-04-22 17:13:19 +0200 |
---|---|---|
committer | Pierre-Marie de Rodat <derodat@adacore.com> | 2021-07-05 13:09:08 +0000 |
commit | 767f9cb1f81bdeb55a7b5fbbccb3c057f37e2fae (patch) | |
tree | ef3594217d93400b11f1948b7734c54ddb17c480 /gcc | |
parent | afe5959e11727f6ff276bc1a7c62181306157504 (diff) | |
download | gcc-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.
Diffstat (limited to 'gcc')
-rw-r--r-- | gcc/ada/sem_ch3.adb | 26 |
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 |