[gcc r12-1585] [Ada] Check all pairs of actuals for overlapping

[gcc r12-1585] [Ada] Check all pairs of actuals for overlapping

[Pierre-Marie de Rodat]

https://gcc.gnu.org/g:a6d844cd860880a9d1f028876085a8d1230819b1

commit r12-1585-ga6d844cd860880a9d1f028876085a8d1230819b1
Author: Piotr Trojanek <trojanek@adacore.com>
Date:   Wed Mar 10 09:26:18 2021 +0100

    [Ada] Check all pairs of actuals for overlapping
    
    gcc/ada/
    
            * sem_warn.adb (Warn_On_Overlapping_Actuals): Fix style;
            refactor repeated calls to Nkind; remove early RETURN.

Diff:
---
 gcc/ada/sem_warn.adb | 14 +++++---------
 1 file changed, 5 insertions(+), 9 deletions(-)

diff --git a/gcc/ada/sem_warn.adb b/gcc/ada/sem_warn.adb
index 6db6d2ccfbe..728927433da 100644
--- a/gcc/ada/sem_warn.adb
+++ b/gcc/ada/sem_warn.adb
@@ -3693,10 +3693,10 @@ package body Sem_Warn is
 
       --  Local variables
 
-      Act1      : Node_Id;
-      Act2      : Node_Id;
-      Form1     : Entity_Id;
-      Form2     : Entity_Id;
+      Act1  : Node_Id;
+      Act2  : Node_Id;
+      Form1 : Entity_Id;
+      Form2 : Entity_Id;
 
    --  Start of processing for Warn_On_Overlapping_Actuals
 
@@ -3708,9 +3708,7 @@ package body Sem_Warn is
 
       --  Exclude calls rewritten as enumeration literals
 
-      if Nkind (N) not in N_Subprogram_Call
-        and then Nkind (N) /= N_Entry_Call_Statement
-      then
+      if Nkind (N) not in N_Subprogram_Call | N_Entry_Call_Statement then
          return;
       end if;
 
@@ -3879,8 +3877,6 @@ package body Sem_Warn is
                      end if;
                   end if;
                end if;
-
-               return;
             end if;
 
             Next_Formal (Form2);