sem_attr.adb (Analyze_Attribute): Factor out heavily indented code in Denote_Same_Fun...
authorHristian Kirtchev <kirtchev@adacore.com>
Mon, 2 Mar 2015 11:09:11 +0000 (11:09 +0000)
committerArnaud Charlet <charlet@gcc.gnu.org>
Mon, 2 Mar 2015 11:09:11 +0000 (12:09 +0100)
2015-03-02  Hristian Kirtchev  <kirtchev@adacore.com>

* sem_attr.adb (Analyze_Attribute): Factor out heavily indented
code in Denote_Same_Function.  Do not analyze attribute 'Result
when it is inside procedure _Postconditions.  Remove a misplaced
warning diagnostic. Code cleanup.
(Denote_Same_Function): New routine.
* sem_prag.adb (Analyze_Contract_Cases_In_Decl_Part): Code
cleanup. Warn on pre/postconditions on an inlined subprogram.
(Analyze_Pragma, Refined_Post case): Warn on pre/postconditions on
an inlined subprogram.
(Analyze_Pre_Post_Condition_In_Decl_Part): Code cleanup. Warn on
pre/post condition on an inlined subprogram.
(Analyze_Test_Case_In_Decl_Part): Code cleanup. Warn on
pre/postconditions on an inlined subprogram.
(Check_Postcondition_Use_In_Inlined_Subprogram): New routine.

From-SVN: r221112

gcc/ada/ChangeLog
gcc/ada/sem_attr.adb
gcc/ada/sem_prag.adb

index b1bab66..1c8ef6a 100644 (file)
@@ -1,5 +1,22 @@
 2015-03-02  Hristian Kirtchev  <kirtchev@adacore.com>
 
+       * sem_attr.adb (Analyze_Attribute): Factor out heavily indented
+       code in Denote_Same_Function.  Do not analyze attribute 'Result
+       when it is inside procedure _Postconditions.  Remove a misplaced
+       warning diagnostic. Code cleanup.
+       (Denote_Same_Function): New routine.
+       * sem_prag.adb (Analyze_Contract_Cases_In_Decl_Part): Code
+       cleanup. Warn on pre/postconditions on an inlined subprogram.
+       (Analyze_Pragma, Refined_Post case): Warn on pre/postconditions on
+       an inlined subprogram.
+       (Analyze_Pre_Post_Condition_In_Decl_Part): Code cleanup. Warn on
+       pre/post condition on an inlined subprogram.
+       (Analyze_Test_Case_In_Decl_Part): Code cleanup. Warn on
+       pre/postconditions on an inlined subprogram.
+       (Check_Postcondition_Use_In_Inlined_Subprogram): New routine.
+
+2015-03-02  Hristian Kirtchev  <kirtchev@adacore.com>
+
        * sem_prag.adb (Ensure_Aggregate_Form):
        Ensure that the name denoted by the Chars of a pragma argument
        association has the proper Sloc when converted into an aggregate.
index 06b9090..b3786f1 100644 (file)
@@ -4997,6 +4997,12 @@ package body Sem_Attr is
          --  Verify that attribute 'Result appears within the Ensures argument
          --  of pragma Test_Case.
 
+         function Denote_Same_Function
+           (Pref_Id : Entity_Id;
+            Spec_Id : Entity_Id) return Boolean;
+         --  Determine whether the entity of the prefix Pref_Id denotes the
+         --  same entity as that of the related subprogram Spec_Id.
+
          function Is_Within
            (Nod      : Node_Id;
             Encl_Nod : Node_Id) return Boolean;
@@ -5078,6 +5084,57 @@ package body Sem_Attr is
             end if;
          end Check_Placement_In_Test_Case;
 
+         --------------------------
+         -- Denote_Same_Function --
+         --------------------------
+
+         function Denote_Same_Function
+           (Pref_Id : Entity_Id;
+            Spec_Id : Entity_Id) return Boolean
+         is
+            Subp_Spec : constant Node_Id := Parent (Spec_Id);
+
+         begin
+            --  The prefix denotes the related subprogram
+
+            if Pref_Id = Spec_Id then
+               return True;
+
+            --  Account for a special case when attribute 'Result appears in
+            --  the postcondition of a generic function.
+
+            --    generic
+            --    function Gen_Func return ...
+            --      with Post => Gen_Func'Result ...;
+
+            --  When the generic function is instantiated, the Chars field of
+            --  the instantiated prefix still denotes the name of the generic
+            --  function. Note that any preemptive transformation is impossible
+            --  without a proper analysis. The structure of the wrapper package
+            --  is as follows:
+
+            --    package Anon_Gen_Pack is
+            --       <subtypes and renamings>
+            --       function Subp_Decl return ...;               --  (!)
+            --       pragma Postcondition (Gen_Func'Result ...);  --  (!)
+            --       function Gen_Func ... renames Subp_Decl;
+            --    end Anon_Gen_Pack;
+
+            elsif Nkind (Subp_Spec) = N_Function_Specification
+              and then Present (Generic_Parent (Subp_Spec))
+              and then Ekind (Pref_Id) = E_Function
+              and then Present (Alias (Pref_Id))
+              and then Alias (Pref_Id) = Spec_Id
+            then
+               return True;
+
+            --  Otherwise the prefix does not denote the related subprogram
+
+            else
+               return False;
+            end if;
+         end Denote_Same_Function;
+
          ---------------
          -- Is_Within --
          ---------------
@@ -5108,14 +5165,11 @@ package body Sem_Attr is
 
          --  Local variables
 
-         In_Post   : Boolean := False;
          Prag      : Node_Id;
          Prag_Id   : Pragma_Id;
          Pref_Id   : Entity_Id;
          Spec_Id   : Entity_Id;
          Subp_Decl : Node_Id;
-         Subp_Id   : Entity_Id;
-         Subp_Spec : Node_Id;
 
       --  Start of processing for Result
 
@@ -5204,128 +5258,62 @@ package body Sem_Attr is
             Subp_Decl := Find_Related_Subprogram_Or_Body (Prag);
          end if;
 
-         --  The pragma where attribute 'Result appears is associated with a
-         --  subprogram declaration or a body.
+         --  The pragma where attribute 'Result resides should be associated
+         --  with a subprogram declaration or a body. If this is not the case,
+         --  then the pragma is illegal. Return as analysis cannot be carried
+         --  out.
 
-         if Nkind_In (Subp_Decl, N_Abstract_Subprogram_Declaration,
-                                 N_Entry_Declaration,
-                                 N_Generic_Subprogram_Declaration,
-                                 N_Subprogram_Body,
-                                 N_Subprogram_Body_Stub,
-                                 N_Subprogram_Declaration)
+         if not Nkind_In (Subp_Decl, N_Abstract_Subprogram_Declaration,
+                                     N_Entry_Declaration,
+                                     N_Generic_Subprogram_Declaration,
+                                     N_Subprogram_Body,
+                                     N_Subprogram_Body_Stub,
+                                     N_Subprogram_Declaration)
          then
-            Subp_Id := Defining_Entity (Subp_Decl);
+            return;
+         end if;
 
-            --  Attribute 'Result is part of the _Postconditions procedure of
-            --  the related subprogram. Retrieve the related subprogram.
+         Spec_Id := Corresponding_Spec_Of (Subp_Decl);
 
-            if Chars (Subp_Id) = Name_uPostconditions then
-               In_Post   := True;
-               Subp_Decl := Parent (Subp_Decl);
-               Subp_Id   := Scope (Subp_Id);
-            end if;
+         --  Attribute 'Result is part of a _Postconditions procedure. There is
+         --  no need to perform the semantic checks below as they were already
+         --  verified when the attribute was analyzed in its original context.
+         --  Instead, rewrite the attribute as a reference to formal parameter
+         --  _Result of the _Postconditions procedure.
 
-            --  Retrieve the entity of the spec (if any)
+         if Chars (Spec_Id) = Name_uPostconditions then
+            Rewrite (N, Make_Identifier (Loc, Name_uResult));
 
-            if Nkind (Subp_Decl) = N_Subprogram_Body
-              and then Present (Corresponding_Spec (Subp_Decl))
-            then
-               Spec_Id := Corresponding_Spec (Subp_Decl);
+            --  The type of formal parameter _Result is that of the function
+            --  encapsulating the _Postconditions procedure. Resolution must
+            --  be carried out against the function return type.
 
-            elsif Nkind (Subp_Decl) = N_Subprogram_Body_Stub
-              and then Present (Corresponding_Spec_Of_Stub (Subp_Decl))
-            then
-               Spec_Id := Corresponding_Spec_Of_Stub (Subp_Decl);
+            Analyze_And_Resolve (N, Etype (Scope (Spec_Id)));
 
-            else
-               Spec_Id := Subp_Id;
-            end if;
+         --  Otherwise attribute 'Result appears in its original context and
+         --  all semantic checks should be carried out.
 
-            --  When the subprogram is always inlined, the postcondition will
-            --  not be propagated to the expanded body.
-
-            if Warn_On_Redundant_Constructs
-              and then Has_Pragma_Inline_Always (Spec_Id)
-            then
-               Error_Msg_N
-                 ("postconditions on inlined functions not enforced?r?", P);
-            end if;
-
-            --  Ensure that the prefix of attribute 'Result denotes the related
-            --  subprogram.
+         else
+            --  Verify the legality of the prefix. It must denotes the entity
+            --  of the related [generic] function.
 
             if Is_Entity_Name (P) then
                Pref_Id := Entity (P);
 
-               --  When a subprogram with contract assertions is imported, it
-               --  is encapsulated in a wrapper. In this case the scope of the
-               --  wrapper denotes the original imported subprogram.
-
-               if Is_Imported (Pref_Id) then
-                  Pref_Id := Scope (Pref_Id);
-               end if;
-
                if Ekind_In (Pref_Id, E_Function, E_Generic_Function) then
+                  if Denote_Same_Function (Pref_Id, Spec_Id) then
+                     Set_Etype (N, Etype (Spec_Id));
 
-                  --  The prefix of attribute 'Result denotes the entity of
-                  --  some other unrelated function.
-
-                  if Pref_Id /= Spec_Id then
-                     Subp_Spec := Parent (Spec_Id);
-
-                     --  Attribute 'Result appears in a postcondition of a
-                     --  generic function that acts as a compilation unit:
-
-                     --    generic
-                     --    function Gen_Func return ...
-                     --      with Post => Gen_Func'Result ...;
-
-                     --  When the function is instantiated, the Chars field of
-                     --  attribute 'Result's prefix still denotes the generic
-                     --  function. Note that any preemptive transformation is
-                     --  impossible without a proper analysis. The structure of
-                     --  the anonymous wrapper package is as follows:
-
-                     --    package Anon_Gen_Pack is
-                     --       <subtypes and renamings>
-                     --       function Subp_Decl return ...;
-                     --       pragma Postcondition (Gen_Func'Result ...);
-                     --       function Gen_Func ... renames Subp_Decl;
-                     --    end Anon_Gen_Pack;
-
-                     --  Recognize this case and do not flag it as illegal
+                  --  Otherwise the prefix denotes some unrelated function
 
-                     if Nkind (Subp_Spec) = N_Function_Specification
-                       and then Present (Generic_Parent (Subp_Spec))
-                     then
-                        if Generic_Parent (Subp_Spec) = Pref_Id then
-                           null;
-
-                        elsif Ekind (Pref_Id) = E_Function
-                          and then Present (Alias (Pref_Id))
-                          and then Alias (Pref_Id) = Spec_Id
-                        then
-                           null;
-
-                        else
-                           Error_Msg_Name_2 := Chars (Spec_Id);
-                           Error_Attr
-                             ("incorrect prefix for % attribute, expected %",
-                              P);
-                        end if;
-
-                     --  Otherwise the context is not a function instantiation
-                     --  and the prefix is illegal
-
-                     else
-                        Error_Msg_Name_2 := Chars (Spec_Id);
-                        Error_Attr
-                          ("incorrect prefix for % attribute, expected %", P);
-                     end if;
+                  else
+                     Error_Msg_Name_2 := Chars (Spec_Id);
+                     Error_Attr
+                       ("incorrect prefix for % attribute, expected %", P);
                   end if;
 
-               --  Otherwise the attribute's prefix denotes some other form of
-               --  a non-function subprogram.
+               --  Otherwise the prefix denotes some other form of subprogram
+               --  entity.
 
                else
                   Error_Attr
@@ -5339,20 +5327,6 @@ package body Sem_Attr is
                Error_Msg_Name_2 := Chars (Spec_Id);
                Error_Attr ("incorrect prefix for % attribute, expected %", P);
             end if;
-
-            --  Attribute 'Result is part of the _Postconditions procedure of
-            --  the related subprogram. Rewrite the attribute as a reference to
-            --  the _Result formal parameter of _Postconditions.
-
-            if In_Post then
-               Rewrite (N, Make_Identifier (Loc, Name_uResult));
-               Analyze_And_Resolve (N, Etype (Subp_Id));
-
-            --  Otherwise decorate the attribute
-
-            else
-               Set_Etype (N, Etype (Subp_Id));
-            end if;
          end if;
       end Result;
 
index 2d84303..9e216c6 100644 (file)
@@ -202,6 +202,13 @@ package body Sem_Prag is
    --  _Post, _Invariant, or _Type_Invariant, which are special names used
    --  in identifiers to represent these attribute references.
 
+   procedure Check_Postcondition_Use_In_Inlined_Subprogram
+     (Prag    : Node_Id;
+      Subp_Id : Entity_Id);
+   --  Subsidiary to the analysis of pragmas Contract_Cases, Postcondition,
+   --  Precondition, Refined_Post and Test_Case. Emit a warning when pragma
+   --  Prag is associated with subprogram Subp_Id subject to Inline_Always.
+
    procedure Check_State_And_Constituent_Use
      (States   : Elist_Id;
       Constits : Elist_Id;
@@ -427,6 +434,7 @@ package body Sem_Prag is
 
       All_Cases : Node_Id;
       CCase     : Node_Id;
+      Spec_Id   : Entity_Id;
       Subp_Decl : Node_Id;
       Subp_Id   : Entity_Id;
 
@@ -439,6 +447,7 @@ package body Sem_Prag is
       Set_Analyzed (N);
 
       Subp_Decl := Find_Related_Subprogram_Or_Body (N);
+      Spec_Id   := Corresponding_Spec_Of (Subp_Decl);
       Subp_Id   := Defining_Entity (Subp_Decl);
       All_Cases := Expression (Get_Argument (N, Subp_Id));
 
@@ -455,14 +464,14 @@ package body Sem_Prag is
          --  to subprogram declarations. Skip the installation for subprogram
          --  bodies because the formals are already visible.
 
-         if not In_Open_Scopes (Subp_Id) then
+         if not In_Open_Scopes (Spec_Id) then
             Restore_Scope := True;
-            Push_Scope (Subp_Id);
+            Push_Scope (Spec_Id);
 
-            if Is_Generic_Subprogram (Subp_Id) then
-               Install_Generic_Formals (Subp_Id);
+            if Is_Generic_Subprogram (Spec_Id) then
+               Install_Generic_Formals (Spec_Id);
             else
-               Install_Formals (Subp_Id);
+               Install_Formals (Spec_Id);
             end if;
          end if;
 
@@ -472,6 +481,11 @@ package body Sem_Prag is
             Next (CCase);
          end loop;
 
+         --  Currently it is not possible to inline pre/postconditions on a
+         --  subprogram subject to pragma Inline_Always.
+
+         Check_Postcondition_Use_In_Inlined_Subprogram (N, Spec_Id);
+
          if Restore_Scope then
             End_Scope;
          end if;
@@ -18465,6 +18479,11 @@ package body Sem_Prag is
             if Legal then
                Analyze_Pre_Post_Condition_In_Decl_Part (N);
 
+               --  Currently it is not possible to inline pre/postconditions on
+               --  a subprogram subject to pragma Inline_Always.
+
+               Check_Postcondition_Use_In_Inlined_Subprogram (N, Spec_Id);
+
                --  Chain the pragma on the contract for easy retrieval
 
                Add_Contract_Item (N, Body_Id);
@@ -21513,6 +21532,11 @@ package body Sem_Prag is
          Process_Class_Wide_Condition (Expr, Spec_Id, Subp_Decl);
       end if;
 
+      --  Currently it is not possible to inline pre/postconditions on a
+      --  subprogram subject to pragma Inline_Always.
+
+      Check_Postcondition_Use_In_Inlined_Subprogram (N, Spec_Id);
+
       --  Remove the subprogram from the scope stack now that the pre-analysis
       --  of the precondition/postcondition is done.
 
@@ -24151,10 +24175,10 @@ package body Sem_Prag is
    procedure Analyze_Test_Case_In_Decl_Part (N : Node_Id) is
       procedure Preanalyze_Test_Case_Arg
         (Arg_Nam : Name_Id;
-         Subp_Id : Entity_Id);
+         Spec_Id : Entity_Id);
       --  Preanalyze one of the optional arguments "Requires" or "Ensures"
-      --  denoted by Arg_Nam. Subp_Id is the entity of the subprogram subject
-      --  to pragma Test_Case.
+      --  denoted by Arg_Nam. Spec_Id is the entity of the subprogram spec
+      --  subject to pragma Test_Case.
 
       ------------------------------
       -- Preanalyze_Test_Case_Arg --
@@ -24162,7 +24186,7 @@ package body Sem_Prag is
 
       procedure Preanalyze_Test_Case_Arg
         (Arg_Nam : Name_Id;
-         Subp_Id : Entity_Id)
+         Spec_Id : Entity_Id)
       is
          Arg : Node_Id;
 
@@ -24170,7 +24194,7 @@ package body Sem_Prag is
          --  Preanalyze the original aspect argument for ASIS or for a generic
          --  subprogram to properly capture global references.
 
-         if ASIS_Mode or else Is_Generic_Subprogram (Subp_Id) then
+         if ASIS_Mode or else Is_Generic_Subprogram (Spec_Id) then
             Arg :=
               Test_Case_Arg
                 (Prag        => N,
@@ -24192,8 +24216,8 @@ package body Sem_Prag is
 
       --  Local variables
 
+      Spec_Id   : Entity_Id;
       Subp_Decl : Node_Id;
-      Subp_Id   : Entity_Id;
 
       Restore_Scope : Boolean := False;
       --  Gets set True if we do a Push_Scope needing a Pop_Scope on exit
@@ -24202,25 +24226,30 @@ package body Sem_Prag is
 
    begin
       Subp_Decl := Find_Related_Subprogram_Or_Body (N);
-      Subp_Id   := Defining_Entity (Subp_Decl);
+      Spec_Id   := Corresponding_Spec_Of (Subp_Decl);
 
       --  Ensure that the formal parameters are visible when analyzing all
       --  clauses. This falls out of the general rule of aspects pertaining
       --  to subprogram declarations.
 
-      if not In_Open_Scopes (Subp_Id) then
+      if not In_Open_Scopes (Spec_Id) then
          Restore_Scope := True;
-         Push_Scope (Subp_Id);
+         Push_Scope (Spec_Id);
 
-         if Is_Generic_Subprogram (Subp_Id) then
-            Install_Generic_Formals (Subp_Id);
+         if Is_Generic_Subprogram (Spec_Id) then
+            Install_Generic_Formals (Spec_Id);
          else
-            Install_Formals (Subp_Id);
+            Install_Formals (Spec_Id);
          end if;
       end if;
 
-      Preanalyze_Test_Case_Arg (Name_Requires, Subp_Id);
-      Preanalyze_Test_Case_Arg (Name_Ensures,  Subp_Id);
+      Preanalyze_Test_Case_Arg (Name_Requires, Spec_Id);
+      Preanalyze_Test_Case_Arg (Name_Ensures,  Spec_Id);
+
+      --  Currently it is not possible to inline pre/postconditions on a
+      --  subprogram subject to pragma Inline_Always.
+
+      Check_Postcondition_Use_In_Inlined_Subprogram (N, Spec_Id);
 
       if Restore_Scope then
          End_Scope;
@@ -24604,6 +24633,32 @@ package body Sem_Prag is
       end if;
    end Check_Missing_Part_Of;
 
+   ---------------------------------------------------
+   -- Check_Postcondition_Use_In_Inlined_Subprogram --
+   ---------------------------------------------------
+
+   procedure Check_Postcondition_Use_In_Inlined_Subprogram
+     (Prag    : Node_Id;
+      Subp_Id : Entity_Id)
+   is
+   begin
+      if Warn_On_Redundant_Constructs
+        and then Has_Pragma_Inline_Always (Subp_Id)
+      then
+         Error_Msg_Name_1 := Original_Aspect_Pragma_Name (Prag);
+
+         if From_Aspect_Specification (Prag) then
+            Error_Msg_NE
+              ("aspect % not enforced on inlined subprogram &?r?",
+               Corresponding_Aspect (Prag), Subp_Id);
+         else
+            Error_Msg_NE
+              ("pragma % not enforced on inlined subprogram &?r?",
+               Prag, Subp_Id);
+         end if;
+      end if;
+   end Check_Postcondition_Use_In_Inlined_Subprogram;
+
    -------------------------------------
    -- Check_State_And_Constituent_Use --
    -------------------------------------