[multiple changes]
authorArnaud Charlet <charlet@gcc.gnu.org>
Wed, 27 May 2015 13:21:36 +0000 (15:21 +0200)
committerArnaud Charlet <charlet@gcc.gnu.org>
Wed, 27 May 2015 13:21:36 +0000 (15:21 +0200)
2015-05-27  Ed Schonberg  <schonberg@adacore.com>

* sem_ch6.adb (Process_Formals): A non-private formal type that
is a limited view does not have a list of private dependents.

2015-05-27  Ed Schonberg  <schonberg@adacore.com>

* exp_ch5.adb (Expand_N_Case_Statement): If the expression in
the case statement is a compile-time known value, we look for a
corresponding alternative to optimize the case statement into a
single case. If the type has a static predicate and the expression
does not satisfy the predicate, there is no legal alternative and
this optimization is not applicable.  Excecution is erroneous,
or else if assertions are enabled, an exception will be raised
earlier, at the point the expression is elaborated.

2015-05-27  Robert Dewar  <dewar@adacore.com>

* sem_elab.adb (Check_Internal_Call_Continue): Suppress
warning on Finalize, Adjust, or Initialize if type involved has
Warnings_Off set.

2015-05-27  Ed Schonberg  <schonberg@adacore.com>

* sem_aux.adb, sem_aux.ads (First_Discriminant): Return empty when
applied to a type with no known discriminants.

From-SVN: r223752

gcc/ada/ChangeLog
gcc/ada/exp_ch5.adb
gcc/ada/sem_aux.adb
gcc/ada/sem_aux.ads
gcc/ada/sem_ch6.adb
gcc/ada/sem_elab.adb

index 0bce664d3d2549ef0576575520d11fd2482eb75e..9d5222b95ce50779e1ea0a67e23b39b070e1ffd7 100644 (file)
@@ -1,3 +1,30 @@
+2015-05-27  Ed Schonberg  <schonberg@adacore.com>
+
+       * sem_ch6.adb (Process_Formals): A non-private formal type that
+       is a limited view does not have a list of private dependents.
+
+2015-05-27  Ed Schonberg  <schonberg@adacore.com>
+
+       * exp_ch5.adb (Expand_N_Case_Statement): If the expression in
+       the case statement is a compile-time known value, we look for a
+       corresponding alternative to optimize the case statement into a
+       single case. If the type has a static predicate and the expression
+       does not satisfy the predicate, there is no legal alternative and
+       this optimization is not applicable.  Excecution is erroneous,
+       or else if assertions are enabled, an exception will be raised
+       earlier, at the point the expression is elaborated.
+
+2015-05-27  Robert Dewar  <dewar@adacore.com>
+
+       * sem_elab.adb (Check_Internal_Call_Continue): Suppress
+       warning on Finalize, Adjust, or Initialize if type involved has
+       Warnings_Off set.
+
+2015-05-27  Ed Schonberg  <schonberg@adacore.com>
+
+       * sem_aux.adb, sem_aux.ads (First_Discriminant): Return empty when
+       applied to a type with no known discriminants.
+
 2015-05-26  Robert Dewar  <dewar@adacore.com>
 
        * errout.ads, sem_ch4.adb, sem_ch6.adb: Minor reformatting.
index ca6971e0530847bf1b085c0f4c697068ad957018..7156c76a8ef2eea63d8917d0aa1050f853abc019 100644 (file)
@@ -2586,9 +2586,16 @@ package body Exp_Ch5 is
 
    begin
       --  Check for the situation where we know at compile time which branch
-      --  will be taken
+      --  will be taken.
 
-      if Compile_Time_Known_Value (Expr) then
+      --  If the value is static but its subtype is predicated and the value
+      --  does not obey the predicate, the value is marked non-static, and
+      --  there can be no corresponding static alternative.
+
+      if Compile_Time_Known_Value (Expr)
+        and then (not Has_Predicates (Etype (Expr))
+                   or else Is_Static_Expression (Expr))
+      then
          Alt := Find_Static_Alternative (N);
 
          --  Do not consider controlled objects found in a case statement which
index 31644b076e36ad9032e22ffc0aa0e1c210c88600..32d5b1f299cdca58999d12a66870fe97799910ac 100644 (file)
@@ -246,7 +246,12 @@ package body Sem_Aux is
          Ent := Next_Entity (Ent);
       end loop;
 
-      pragma Assert (Ekind (Ent) = E_Discriminant);
+      --  Call may be on a private type with unknown discriminants, in which
+      --  case Ent is Empty, and as per the spec, we return Empty in this case.
+
+      --  Historical note: The revious assertion that Ent is a discriminant
+      --  was overly cautious and prevented application of this function in
+      --  SPARK applications.
 
       return Ent;
    end First_Discriminant;
index 5268b011a3a62880c6ef85b4ea9f372208053f93..db0931e0713588486fb553225facce5da1c379bd 100644 (file)
@@ -119,9 +119,9 @@ package Sem_Aux is
    --  First_Entity. The exception arises for tagged types, where the tag
    --  itself is prepended to the front of the entity chain, so the
    --  First_Discriminant function steps past the tag if it is present.
-   --  The caller is responsible for checking that the type has discriminants,
-   --  so for example it is improper to call this function on a private
-   --  type with unknown discriminants.
+   --  The caller is responsible for checking that the type has discriminants.
+   --  When called on a private type with unknown discriminants, the function
+   --  always returns Empty.
 
    function First_Stored_Discriminant (Typ : Entity_Id) return Entity_Id;
    --  Typ is a type with discriminants. Gives the first discriminant stored
index 43cbffce8be4ea0a191fe9f8ca6b8cf760a2c399..18a9b0299ee80be15cc7a18f0e3b5ebd76326dac 100644 (file)
@@ -10117,9 +10117,13 @@ package body Sem_Ch6 is
                        (Parent (T), N_Access_Function_Definition,
                                     N_Access_Procedure_Definition)
                      then
-                        if not Is_Class_Wide_Type (Formal_Type) then
+                        --  A limited view has no private dependents
+
+                        if not Is_Class_Wide_Type (Formal_Type)
+                          and then not From_Limited_With (Formal_Type)
+                        then
                            Append_Elmt (Current_Scope,
-                               Private_Dependents (Base_Type (Formal_Type)));
+                             Private_Dependents (Base_Type (Formal_Type)));
                         end if;
 
                         --  Freezing is delayed to ensure that Register_Prim
index 07517bbc467601e950e78d8e84af8201b0a31184..01fd0cd969e0fe1d4d3cbf1e875267db54ceb5cc 100644 (file)
@@ -2447,6 +2447,30 @@ package body Sem_Elab is
                  ("instantiation of& may occur before body is seen<l<",
                   N, Orig_Ent);
             else
+               --  A rather specific check. For Finalize/Adjust/Initialize,
+               --  if the type has Warnings_Off set, suppress the warning.
+
+               if Nam_In (Chars (E), Name_Adjust,
+                                     Name_Finalize,
+                                     Name_Initialize)
+                 and then Present (First_Formal (E))
+               then
+                  declare
+                     T : constant Entity_Id := Etype (First_Formal (E));
+                  begin
+                     if Is_Controlled (T) then
+                        if Warnings_Off (T)
+                          or else (Ekind (T) = E_Private_Type
+                                    and then Warnings_Off (Full_View (T)))
+                        then
+                           goto Output;
+                        end if;
+                     end if;
+                  end;
+               end if;
+
+               --  Go ahead and give warning if not this special case
+
                Error_Msg_NE
                  ("call to& may occur before body is seen<l<", N, Orig_Ent);
             end if;
@@ -2458,6 +2482,8 @@ package body Sem_Elab is
             --  all the clarification messages produces by Output_Calls must be
             --  emitted unconditionally.
 
+            <<Output>>
+
             Output_Calls (N, Check_Elab_Flag => False);
          end if;
       end if;