[multiple changes]
authorArnaud Charlet <charlet@gcc.gnu.org>
Wed, 6 Sep 2017 10:41:57 +0000 (12:41 +0200)
committerArnaud Charlet <charlet@gcc.gnu.org>
Wed, 6 Sep 2017 10:41:57 +0000 (12:41 +0200)
2017-09-06  Bob Duff  <duff@adacore.com>

* a-comlin.ads, a-comlin.adb (Argument): Move the constraint
check back to the body, because SPARK is not yet ready for
"or else raise Constraint_Error".

2017-09-06  Ed Schonberg  <schonberg@adacore.com>

* exp_ch6.adb (Expand_Call_Helper): Replace call to null
procedure by a single null statement, after evaluating the
actuals that require it.

2017-09-06  Javier Miranda  <miranda@adacore.com>

* exp_aggr.adb (Backend_Processing_Possible.Component_Check):
Generating C code improve the code that checks the use of nested
aggregates to initialize object declarations.

2017-09-06  Yannick Moy  <moy@adacore.com>

* sem_ch3.adb (Derived_Type_Declaration): Detect
violations of new rule SPARK RM 3.4(1).  Also refactor existing
check to use the new function Find_Partial_View.

2017-09-06  Vincent Celier  <celier@adacore.com>

* gnatcmd.adb: gnat ls -V -P... invokes gprls -V -P... The code
from the Prj hierarchy has been removed from the GNAT driver.

2017-09-06  Ed Schonberg  <schonberg@adacore.com>

* sem_type.adb (Interface_Present_In_Ancestor): Within an
expression function, or within a spec expression (default value,
etc) a reference to an incomplete type is legal: legality of
the operation will be checked when some related entity (type,
object or subprogram) is frozen.

From-SVN: r251776

gcc/ada/ChangeLog
gcc/ada/a-comlin.adb
gcc/ada/a-comlin.ads
gcc/ada/exp_aggr.adb
gcc/ada/exp_ch6.adb
gcc/ada/gnatcmd.adb
gcc/ada/sem_ch3.adb
gcc/ada/sem_type.adb

index 0d3f844..7a146be 100644 (file)
@@ -1,3 +1,40 @@
+2017-09-06  Bob Duff  <duff@adacore.com>
+
+       * a-comlin.ads, a-comlin.adb (Argument): Move the constraint
+       check back to the body, because SPARK is not yet ready for
+       "or else raise Constraint_Error".
+
+2017-09-06  Ed Schonberg  <schonberg@adacore.com>
+
+       * exp_ch6.adb (Expand_Call_Helper): Replace call to null
+       procedure by a single null statement, after evaluating the
+       actuals that require it.
+
+2017-09-06  Javier Miranda  <miranda@adacore.com>
+
+       * exp_aggr.adb (Backend_Processing_Possible.Component_Check):
+       Generating C code improve the code that checks the use of nested
+       aggregates to initialize object declarations.
+
+2017-09-06  Yannick Moy  <moy@adacore.com>
+
+       * sem_ch3.adb (Derived_Type_Declaration): Detect
+       violations of new rule SPARK RM 3.4(1).  Also refactor existing
+       check to use the new function Find_Partial_View.
+
+2017-09-06  Vincent Celier  <celier@adacore.com>
+
+       * gnatcmd.adb: gnat ls -V -P... invokes gprls -V -P...  The code
+       from the Prj hierarchy has been removed from the GNAT driver.
+
+2017-09-06  Ed Schonberg  <schonberg@adacore.com>
+
+       * sem_type.adb (Interface_Present_In_Ancestor): Within an
+       expression function, or within a spec expression (default value,
+       etc) a reference to an incomplete type is legal: legality of
+       the operation will be checked when some related entity (type,
+       object or subprogram) is frozen.
+
 2017-09-06  Gary Dismukes  <dismukes@adacore.com>
 
        * exp_ch5.adb, s-diinio.ads, sem_ch4.adb, s-diflio.ads: Minor spelling
index 2af8bd9..49caca5 100644 (file)
@@ -56,12 +56,19 @@ package body Ada.Command_Line is
    --------------
 
    function Argument (Number : Positive) return String is
-      Num : constant Positive :=
-        (if Remove_Args = null then Number else Remove_Args (Number));
-      Arg : aliased String (1 .. Len_Arg (Num));
    begin
-      Fill_Arg (Arg'Address, Num);
-      return Arg;
+      if Number > Argument_Count then
+         raise Constraint_Error;
+      end if;
+
+      declare
+         Num : constant Positive :=
+           (if Remove_Args = null then Number else Remove_Args (Number));
+         Arg : aliased String (1 .. Len_Arg (Num));
+      begin
+         Fill_Arg (Arg'Address, Num);
+         return Arg;
+      end;
    end Argument;
 
    --------------------
index b8e5562..c4eecef 100644 (file)
@@ -43,8 +43,14 @@ package Ada.Command_Line is
    --
    --  In GNAT: Corresponds to (argc - 1) in C.
 
+   pragma Assertion_Policy (Pre => Ignore);
+   --  We need to ignore the precondition of Argument, below, so that we don't
+   --  raise Assertion_Error. The body raises Constraint_Error. It would be
+   --  cleaner to add "or else raise Constraint_Error" to the precondition, but
+   --  SPARK does not yet support raise expressions.
+
    function Argument (Number : Positive) return String with
-      Pre => Number <= Argument_Count or else raise Constraint_Error;
+      Pre => Number <= Argument_Count;
    --  If the external execution environment supports passing arguments to
    --  a program, then Argument returns an implementation-defined value
    --  corresponding to the argument at relative position Number. If Number
index 460b1c1..38d233b 100644 (file)
@@ -644,15 +644,27 @@ package body Exp_Aggr is
             return False;
          end if;
 
-         --  Checks 11: (part of an object declaration)
+         --  Checks 11: The C code generator cannot handle aggregates that
+         --  are not part of an object declaration.
 
-         if Modify_Tree_For_C
-           and then Nkind (Parent (N)) /= N_Object_Declaration
-           and then
-             (Nkind (Parent (N)) /= N_Qualified_Expression
-               or else Nkind (Parent (Parent (N))) /= N_Object_Declaration)
-         then
-            return False;
+         if Modify_Tree_For_C then
+            declare
+               Par : Node_Id := Parent (N);
+
+            begin
+               --  Skip enclosing nested aggregates and their qualified
+               --  expressions
+
+               while Nkind (Par) = N_Aggregate
+                 or else Nkind (Par) = N_Qualified_Expression
+               loop
+                  Par := Parent (Par);
+               end loop;
+
+               if Nkind (Par) /= N_Object_Declaration then
+                  return False;
+               end if;
+            end;
          end if;
 
          --  Checks on components
index 756eeab..55831e4 100644 (file)
@@ -3951,6 +3951,20 @@ package body Exp_Ch6 is
               Unchecked_Convert_To
                 (RTE (RE_Address), Relocate_Node (First_Actual (Call_Node))));
             return;
+
+         --  A call to a null procedure is replaced by a null statement, but
+         --  we are not allowed to ignore possible side effects of the call,
+         --  so we make sure that actuals are evaluated.
+
+         elsif Is_Null_Procedure (Subp) then
+            Actual := First_Actual (Call_Node);
+            while Present (Actual) loop
+               Remove_Side_Effects (Actual);
+               Next_Actual (Actual);
+            end loop;
+
+            Rewrite (Call_Node, Make_Null_Statement (Loc));
+            return;
          end if;
 
          --  Handle inlining. No action needed if the subprogram is not inlined
index e82e8d5..e5df7bb 100644 (file)
@@ -6,7 +6,7 @@
 --                                                                          --
 --                                 B o d y                                  --
 --                                                                          --
---          Copyright (C) 1996-2016, Free Software Foundation, Inc.         --
+--          Copyright (C) 1996-2017, Free Software Foundation, Inc.         --
 --                                                                          --
 -- GNAT is free software;  you can  redistribute it  and/or modify it under --
 -- terms of the  GNU General Public License as published  by the Free Soft- --
 --                                                                          --
 ------------------------------------------------------------------------------
 
-with Csets;
 with Gnatvsn;
-with Makeutl;  use Makeutl;
 with Namet;    use Namet;
 with Opt;      use Opt;
 with Osint;    use Osint;
 with Output;   use Output;
-with Prj;      use Prj;
-with Prj.Env;
-with Prj.Ext;  use Prj.Ext;
-with Prj.Pars;
-with Prj.Tree; use Prj.Tree;
-with Prj.Util; use Prj.Util;
-with Sdefault;
-with Sinput.P;
-with Snames;   use Snames;
-with Stringt;
 with Switch;   use Switch;
 with Table;
-with Tempdir;
-with Types;    use Types;
 
 with Ada.Characters.Handling; use Ada.Characters.Handling;
 with Ada.Command_Line;        use Ada.Command_Line;
 with Ada.Text_IO;             use Ada.Text_IO;
 
-with GNAT.Directory_Operations; use GNAT.Directory_Operations;
 with GNAT.OS_Lib;               use GNAT.OS_Lib;
 
 procedure GNATCmd is
@@ -97,25 +82,6 @@ procedure GNATCmd is
       Pp    => Pretty);
    --  Mapping of alternate commands to commands
 
-   Call_GPR_Tool : Boolean := False;
-   --  True when a GPR tool should be called, if available
-
-   Project_Node_Tree : Project_Node_Tree_Ref;
-   Project_File      : String_Access;
-   Project           : Prj.Project_Id;
-   Current_Verbosity : Prj.Verbosity := Prj.Default;
-   Tool_Package_Name : Name_Id       := No_Name;
-
-   Project_Tree : constant Project_Tree_Ref :=
-                    new Project_Tree_Data (Is_Root_Tree => True);
-   --  The project tree
-
-   All_Projects : Boolean := False;
-
-   Temp_File_Name : Path_Name_Type := No_Path;
-   --  The name of the temporary text file to put a list of source/object
-   --  files to pass to a tool.
-
    package First_Switches is new Table.Table
      (Table_Component_Type => String_Access,
       Table_Index_Type     => Integer,
@@ -256,178 +222,16 @@ procedure GNATCmd is
          Unixsws  => null)
      );
 
-   subtype SA is String_Access;
-
-   Naming_String      : constant SA := new String'("naming");
-   Gnatls_String      : constant SA := new String'("gnatls");
-
-   Packages_To_Check_By_Gnatls    : constant String_List_Access :=
-     new String_List'((Naming_String, Gnatls_String));
-
-   Packages_To_Check : String_List_Access := Prj.All_Packages;
-
    -----------------------
    -- Local Subprograms --
    -----------------------
 
-   procedure Check_Files;
-   --  For GNAT LIST -V, check if a project file is specified, without any file
-   --  arguments and without a switch -files=. If it is the case, invoke the
-   --  GNAT tool with the proper list of files, derived from the sources of
-   --  the project.
-
    procedure Output_Version;
    --  Output the version of this program
 
    procedure Usage;
    --  Display usage
 
-   -----------------
-   -- Check_Files --
-   -----------------
-
-   procedure Check_Files is
-      Add_Sources : Boolean := True;
-      Unit        : Prj.Unit_Index;
-      Subunit     : Boolean := False;
-      FD          : File_Descriptor := Invalid_FD;
-      Status      : Integer;
-      Success     : Boolean;
-
-      procedure Add_To_Response_File
-        (File_Name  : String;
-         Check_File : Boolean := True);
-      --  Include the file name passed as parameter in the response file for
-      --  the tool being called. If the response file can not be written then
-      --  the file name is passed in the parameter list of the tool. If the
-      --  Check_File parameter is True then the procedure verifies the
-      --  existence of the file before adding it to the response file.
-
-      --------------------------
-      -- Add_To_Response_File --
-      --------------------------
-
-      procedure Add_To_Response_File
-        (File_Name  : String;
-         Check_File : Boolean := True)
-      is
-      begin
-         Name_Len := 0;
-
-         Add_Str_To_Name_Buffer (File_Name);
-
-         if not Check_File or else
-           Is_Regular_File (Name_Buffer (1 .. Name_Len))
-         then
-            if FD /= Invalid_FD then
-               Name_Len := Name_Len + 1;
-               Name_Buffer (Name_Len) := ASCII.LF;
-
-               Status := Write (FD, Name_Buffer (1)'Address, Name_Len);
-
-               if Status /= Name_Len then
-                  Osint.Fail ("disk full");
-               end if;
-            else
-               Last_Switches.Increment_Last;
-               Last_Switches.Table (Last_Switches.Last) :=
-                 new String'(File_Name);
-            end if;
-         end if;
-      end Add_To_Response_File;
-
-   --  Start of processing for Check_Files
-
-   begin
-      --  Check if there is at least one argument that is not a switch
-
-      for Index in 1 .. Last_Switches.Last loop
-         if Last_Switches.Table (Index) (1) /= '-'
-           or else (Last_Switches.Table (Index).all'Length > 7
-                     and then Last_Switches.Table (Index) (1 .. 7) = "-files=")
-         then
-            Add_Sources := False;
-            exit;
-         end if;
-      end loop;
-
-      --  If all arguments are switches and there is no switch -files=, add the
-      --  path names of all the sources of the main project.
-
-      if Add_Sources then
-         Tempdir.Create_Temp_File (FD, Temp_File_Name);
-         Record_Temp_File (Project_Tree.Shared, Temp_File_Name);
-         Last_Switches.Increment_Last;
-         Last_Switches.Table (Last_Switches.Last) :=
-           new String'("-files=" & Get_Name_String (Temp_File_Name));
-
-         Unit := Units_Htable.Get_First (Project_Tree.Units_HT);
-         while Unit /= No_Unit_Index loop
-
-            --  We only need to put the library units, body or spec, but not
-            --  the subunits.
-
-            if Unit.File_Names (Impl) /= null
-              and then not Unit.File_Names (Impl).Locally_Removed
-            then
-               --  There is a body, check if it is for this project
-
-               if All_Projects
-                 or else Unit.File_Names (Impl).Project = Project
-               then
-                  Subunit := False;
-
-                  if Unit.File_Names (Spec) = null
-                    or else Unit.File_Names (Spec).Locally_Removed
-                  then
-                     --  We have a body with no spec: we need to check if
-                     --  this is a subunit, because gnatls will complain
-                     --  about subunits.
-
-                     declare
-                        Src_Ind : constant Source_File_Index :=
-                                    Sinput.P.Load_Project_File
-                                      (Get_Name_String
-                                         (Unit.File_Names (Impl).Path.Name));
-                     begin
-                        Subunit := Sinput.P.Source_File_Is_Subunit (Src_Ind);
-                     end;
-                  end if;
-
-                  if not Subunit then
-                     Add_To_Response_File
-                       (Get_Name_String (Unit.File_Names (Impl).Display_File),
-                        Check_File => False);
-                  end if;
-               end if;
-
-            elsif Unit.File_Names (Spec) /= null
-              and then not Unit.File_Names (Spec).Locally_Removed
-            then
-               --  We have a spec with no body. Check if it is for this project
-
-               if All_Projects
-                 or else Unit.File_Names (Spec).Project = Project
-               then
-                  Add_To_Response_File
-                    (Get_Name_String (Unit.File_Names (Spec).Display_File),
-                     Check_File => False);
-               end if;
-            end if;
-
-            Unit := Units_Htable.Get_Next (Project_Tree.Units_HT);
-         end loop;
-
-         if FD /= Invalid_FD then
-            Close (FD, Success);
-
-            if not Success then
-               Osint.Fail ("disk full");
-            end if;
-         end if;
-      end if;
-   end Check_Files;
-
    --------------------
    -- Output_Version --
    --------------------
@@ -485,17 +289,6 @@ begin
 
    --  Initializations
 
-   Csets.Initialize;
-   Snames.Initialize;
-   Stringt.Initialize;
-
-   Prj.Tree.Initialize (Root_Environment, Gnatmake_Flags);
-
-   Project_Node_Tree := new Project_Node_Tree_Data;
-   Prj.Tree.Initialize (Project_Node_Tree);
-
-   Prj.Initialize (Project_Tree);
-
    Last_Switches.Init;
    Last_Switches.Set_Last (0);
 
@@ -679,23 +472,17 @@ begin
            or else The_Command = List
          then
             declare
-               Switch            : String_Access;
-               Dash_V_Switch     : constant String := "-V";
-
+               Switch        : String_Access;
+               Call_GPR_Tool : Boolean := False;
             begin
                for J in 1 .. Last_Switches.Last loop
                   Switch := Last_Switches.Table (J);
 
-                  if The_Command = List and then Switch.all = Dash_V_Switch
-                  then
-                     Call_GPR_Tool := False;
-                     exit;
-                  end if;
-
                   if Switch'Length >= 2
                     and then Switch (Switch'First .. Switch'First + 1) = "-P"
                   then
                      Call_GPR_Tool := True;
+                     exit;
                   end if;
                end loop;
 
@@ -806,390 +593,6 @@ begin
          end;
       end if;
 
-      if The_Command = List and then not Call_GPR_Tool then
-         Tool_Package_Name := Name_Gnatls;
-         Packages_To_Check := Packages_To_Check_By_Gnatls;
-
-         --  Check that the switches are consistent. Detect project file
-         --  related switches.
-
-         Inspect_Switches : declare
-            Arg_Num : Positive := 1;
-            Argv    : String_Access;
-
-            procedure Remove_Switch (Num : Positive);
-            --  Remove a project related switch from table Last_Switches
-
-            -------------------
-            -- Remove_Switch --
-            -------------------
-
-            procedure Remove_Switch (Num : Positive) is
-            begin
-               Last_Switches.Table (Num .. Last_Switches.Last - 1) :=
-                 Last_Switches.Table (Num + 1 .. Last_Switches.Last);
-               Last_Switches.Decrement_Last;
-            end Remove_Switch;
-
-         --  Start of processing for Inspect_Switches
-
-         begin
-            while Arg_Num <= Last_Switches.Last loop
-               Argv := Last_Switches.Table (Arg_Num);
-
-               if Argv (Argv'First) = '-' then
-                  if Argv'Length = 1 then
-                     Fail ("switch character cannot be followed by a blank");
-                  end if;
-
-                  --  --subdirs=... Specify Subdirs
-
-                  if Argv'Length > Makeutl.Subdirs_Option'Length
-                    and then
-                      Argv
-                       (Argv'First ..
-                        Argv'First + Makeutl.Subdirs_Option'Length - 1) =
-                                                        Makeutl.Subdirs_Option
-                  then
-                     Subdirs :=
-                       new String'
-                         (Argv (Argv'First + Makeutl.Subdirs_Option'Length ..
-                                Argv'Last));
-
-                     Remove_Switch (Arg_Num);
-
-                  --  -aPdir  Add dir to the project search path
-
-                  elsif Argv'Length > 3
-                    and then Argv (Argv'First + 1 .. Argv'First + 2) = "aP"
-                  then
-                     Prj.Env.Add_Directories
-                       (Root_Environment.Project_Path,
-                        Argv (Argv'First + 3 .. Argv'Last));
-
-                     --  Pass -aPdir to gnatls, but not to other tools
-
-                     if The_Command = List then
-                        Arg_Num := Arg_Num + 1;
-                     else
-                        Remove_Switch (Arg_Num);
-                     end if;
-
-                  --  -eL  Follow links for files
-
-                  elsif Argv.all = "-eL" then
-                     Follow_Links_For_Files := True;
-                     Follow_Links_For_Dirs  := True;
-
-                     Remove_Switch (Arg_Num);
-
-                  --  -vPx  Specify verbosity while parsing project files
-
-                  elsif Argv'Length >= 3
-                    and then  Argv (Argv'First + 1 .. Argv'First + 2) = "vP"
-                  then
-                     if Argv'Length = 4
-                       and then Argv (Argv'Last) in '0' .. '2'
-                     then
-                        case Argv (Argv'Last) is
-                           when '0' =>
-                              Current_Verbosity := Prj.Default;
-                           when '1' =>
-                              Current_Verbosity := Prj.Medium;
-                           when '2' =>
-                              Current_Verbosity := Prj.High;
-                           when others =>
-
-                              --  Cannot happen
-
-                              raise Program_Error;
-                        end case;
-                     else
-                        Fail ("invalid verbosity level: "
-                              & Argv (Argv'First + 3 .. Argv'Last));
-                     end if;
-
-                     Remove_Switch (Arg_Num);
-
-                  --  -Pproject_file  Specify project file to be used
-
-                  elsif Argv (Argv'First + 1) = 'P' then
-
-                     --  Only one -P switch can be used
-
-                     if Project_File /= null then
-                        Fail
-                          (Argv.all
-                           & ": second project file forbidden (first is """
-                           & Project_File.all & """)");
-
-                     elsif Argv'Length = 2 then
-
-                        --  There is space between -P and the project file
-                        --  name. -P cannot be the last option.
-
-                        if Arg_Num = Last_Switches.Last then
-                           Fail ("project file name missing after -P");
-
-                        else
-                           Remove_Switch (Arg_Num);
-                           Argv := Last_Switches.Table (Arg_Num);
-
-                           --  After -P, there must be a project file name,
-                           --  not another switch.
-
-                           if Argv (Argv'First) = '-' then
-                              Fail ("project file name missing after -P");
-
-                           else
-                              Project_File := new String'(Argv.all);
-                           end if;
-                        end if;
-
-                     else
-                        --  No space between -P and project file name
-
-                        Project_File :=
-                          new String'(Argv (Argv'First + 2 .. Argv'Last));
-                     end if;
-
-                     Remove_Switch (Arg_Num);
-
-                  --  -Xexternal=value Specify an external reference to be
-                  --                   used in project files
-
-                  elsif Argv'Length >= 5
-                    and then Argv (Argv'First + 1) = 'X'
-                  then
-                     if not Check (Root_Environment.External,
-                                    Argv (Argv'First + 2 .. Argv'Last))
-                     then
-                        Fail
-                          (Argv.all & " is not a valid external assignment.");
-                     end if;
-
-                     Remove_Switch (Arg_Num);
-
-                  --  --unchecked-shared-lib-imports
-
-                  elsif Argv.all = "--unchecked-shared-lib-imports" then
-                     Opt.Unchecked_Shared_Lib_Imports := True;
-                     Remove_Switch (Arg_Num);
-
-                  --  gnat list -U
-
-                  elsif
-                    The_Command = List
-                    and then Argv'Length = 2
-                    and then Argv (2) = 'U'
-                  then
-                     All_Projects := True;
-                     Remove_Switch (Arg_Num);
-
-                  else
-                     Arg_Num := Arg_Num + 1;
-                  end if;
-
-               else
-                  Arg_Num := Arg_Num + 1;
-               end if;
-            end loop;
-         end Inspect_Switches;
-      end if;
-
-      --  Add the default project search directories now, after the directories
-      --  that have been specified by switches -aP<dir>.
-
-      Prj.Env.Initialize_Default_Project_Path
-        (Root_Environment.Project_Path,
-         Target_Name => Sdefault.Target_Name.all);
-
-      --  If there is a project file specified, parse it, get the switches
-      --  for the tool and setup PATH environment variables.
-
-      if Project_File /= null then
-         Prj.Pars.Set_Verbosity (To => Current_Verbosity);
-
-         Prj.Pars.Parse
-           (Project           => Project,
-            In_Tree           => Project_Tree,
-            In_Node_Tree      => Project_Node_Tree,
-            Project_File_Name => Project_File.all,
-            Env               => Root_Environment,
-            Packages_To_Check => Packages_To_Check);
-
-         --  Prj.Pars.Parse calls Set_Standard_Output, reset to stderr
-
-         Set_Standard_Error;
-
-         if Project = Prj.No_Project then
-            Fail ("""" & Project_File.all & """ processing failed");
-
-         elsif Project.Qualifier = Aggregate then
-            Fail ("aggregate projects are not supported");
-
-         elsif Aggregate_Libraries_In (Project_Tree) then
-            Fail ("aggregate library projects are not supported");
-         end if;
-
-         --  Check if a package with the name of the tool is in the project
-         --  file and if there is one, get the switches, if any, and scan them.
-
-         declare
-            Pkg : constant Prj.Package_Id :=
-                    Prj.Util.Value_Of
-                      (Name        => Tool_Package_Name,
-                       In_Packages => Project.Decl.Packages,
-                       Shared      => Project_Tree.Shared);
-
-            Element : Package_Element;
-
-            Switches_Array : Array_Element_Id;
-
-            The_Switches : Prj.Variable_Value;
-            Current      : Prj.String_List_Id;
-            The_String   : String_Element;
-
-            Main : String_Access := null;
-
-         begin
-            if Pkg /= No_Package then
-               Element := Project_Tree.Shared.Packages.Table (Pkg);
-
-               --  Package Gnatls has a single attribute Switches, that is not
-               --  an associative array.
-
-               if The_Command = List then
-                  The_Switches :=
-                    Prj.Util.Value_Of
-                    (Variable_Name => Snames.Name_Switches,
-                     In_Variables  => Element.Decl.Attributes,
-                     Shared        => Project_Tree.Shared);
-
-               --  Packages Binder (for gnatbind), Cross_Reference (for
-               --  gnatxref), Linker (for gnatlink), Finder (for gnatfind),
-               --  have an attributed Switches, an associative array, indexed
-               --  by the name of the file.
-
-               --  They also have an attribute Default_Switches, indexed by the
-               --  name of the programming language.
-
-               else
-                  --  First check if there is a single main
-
-                  for J in 1 .. Last_Switches.Last loop
-                     if Last_Switches.Table (J) (1) /= '-' then
-                        if Main = null then
-                           Main := Last_Switches.Table (J);
-                        else
-                           Main := null;
-                           exit;
-                        end if;
-                     end if;
-                  end loop;
-
-                  if Main /= null then
-                     Switches_Array :=
-                       Prj.Util.Value_Of
-                         (Name      => Name_Switches,
-                          In_Arrays => Element.Decl.Arrays,
-                          Shared    => Project_Tree.Shared);
-                     Name_Len := 0;
-
-                     --  If the single main has been specified as an absolute
-                     --  path, use only the simple file name. If the absolute
-                     --  path is incorrect, an error will be reported by the
-                     --  underlying tool and it does not make a difference
-                     --  what switches are used.
-
-                     if Is_Absolute_Path (Main.all) then
-                        Add_Str_To_Name_Buffer (File_Name (Main.all));
-                     else
-                        Add_Str_To_Name_Buffer (Main.all);
-                     end if;
-
-                     The_Switches := Prj.Util.Value_Of
-                       (Index     => Name_Find,
-                        Src_Index => 0,
-                        In_Array  => Switches_Array,
-                        Shared    => Project_Tree.Shared);
-                  end if;
-
-                  if The_Switches.Kind = Prj.Undefined then
-                     Switches_Array :=
-                       Prj.Util.Value_Of
-                         (Name      => Name_Default_Switches,
-                          In_Arrays => Element.Decl.Arrays,
-                          Shared    => Project_Tree.Shared);
-                     The_Switches := Prj.Util.Value_Of
-                       (Index     => Name_Ada,
-                        Src_Index => 0,
-                        In_Array  => Switches_Array,
-                        Shared    => Project_Tree.Shared);
-                  end if;
-               end if;
-
-               --  If there are switches specified in the package of the
-               --  project file corresponding to the tool, scan them.
-
-               case The_Switches.Kind is
-                  when Prj.Undefined =>
-                     null;
-
-                  when Prj.Single =>
-                     declare
-                        Switch : constant String :=
-                                   Get_Name_String (The_Switches.Value);
-                     begin
-                        if Switch'Length > 0 then
-                           First_Switches.Increment_Last;
-                           First_Switches.Table (First_Switches.Last) :=
-                             new String'(Switch);
-                        end if;
-                     end;
-
-                  when Prj.List =>
-                     Current := The_Switches.Values;
-                     while Current /= Prj.Nil_String loop
-                        The_String := Project_Tree.Shared.String_Elements.
-                                        Table (Current);
-
-                        declare
-                           Switch : constant String :=
-                                      Get_Name_String (The_String.Value);
-                        begin
-                           if Switch'Length > 0 then
-                              First_Switches.Increment_Last;
-                              First_Switches.Table (First_Switches.Last) :=
-                                new String'(Switch);
-                           end if;
-                        end;
-
-                        Current := The_String.Next;
-                     end loop;
-               end case;
-            end if;
-         end;
-
-         if The_Command = Bind or else The_Command = Link then
-            if Project.Object_Directory.Name = No_Path then
-               Fail ("project " & Get_Name_String (Project.Display_Name)
-                     & " has no object directory");
-            end if;
-
-            Change_Dir (Get_Name_String (Project.Object_Directory.Name));
-         end if;
-
-         --  Set up the env vars for project path files
-
-         Prj.Env.Set_Ada_Paths
-           (Project, Project_Tree, Including_Libraries => True);
-
-         if The_Command = List then
-            Check_Files;
-         end if;
-      end if;
-
       --  Gather all the arguments and invoke the executable
 
       declare
@@ -1220,10 +623,6 @@ begin
 
          My_Exit_Status := Exit_Status (Spawn (Exec_Path.all, The_Args));
 
-         if not Keep_Temporary_Files then
-            Delete_All_Temp_Files (Project_Tree.Shared);
-         end if;
-
          Set_Exit_Status (My_Exit_Status);
       end;
    end;
index b5fb5f9..be241a4 100644 (file)
@@ -16241,6 +16241,9 @@ package body Sem_Ch3 is
       --  Check whether the parent type is a generic formal, or derives
       --  directly or indirectly from one.
 
+      function Find_Partial_View (T : Entity_Id) return Entity_Id;
+      --  Return the partial view for a type entity T, when there is one
+
       ------------------------
       -- Comes_From_Generic --
       ------------------------
@@ -16267,6 +16270,28 @@ package body Sem_Ch3 is
          end if;
       end Comes_From_Generic;
 
+      -----------------------
+      -- Find_Partial_View --
+      -----------------------
+
+      function Find_Partial_View (T : Entity_Id) return Entity_Id is
+         Partial_View : Entity_Id;
+
+      begin
+         --  Look for the associated private type declaration
+
+         Partial_View := First_Entity (Scope (T));
+         loop
+            exit when No (Partial_View)
+              or else (Has_Private_Declaration (Partial_View)
+                        and then Full_View (Partial_View) = T);
+
+            Next_Entity (Partial_View);
+         end loop;
+
+         return Partial_View;
+      end Find_Partial_View;
+
       --  Local variables
 
       Def          : constant Node_Id := Type_Definition (N);
@@ -16281,6 +16306,28 @@ package body Sem_Ch3 is
    begin
       Parent_Type := Find_Type_Of_Subtype_Indic (Indic);
 
+      if SPARK_Mode = On
+        and then Is_Tagged_Type (Parent_Type)
+      then
+         declare
+            Partial_View : constant Entity_Id :=
+              Find_Partial_View (Parent_Type);
+
+         begin
+            --  If the partial view was not found then the parent type is not a
+            --  private type. Otherwise check that the partial view is declared
+            --  as tagged.
+
+            if Present (Partial_View)
+              and then not Is_Tagged_Type (Partial_View)
+            then
+               Error_Msg_NE ("cannot derive from & declared as "
+                             & "untagged private (SPARK RM 3.4(1))",
+                             N, Partial_View);
+            end if;
+         end;
+      end if;
+
       --  Ada 2005 (AI-251): In case of interface derivation check that the
       --  parent is also an interface.
 
@@ -16468,14 +16515,7 @@ package body Sem_Ch3 is
          begin
             --  Look for the associated private type declaration
 
-            Partial_View := First_Entity (Current_Scope);
-            loop
-               exit when No (Partial_View)
-                 or else (Has_Private_Declaration (Partial_View)
-                           and then Full_View (Partial_View) = T);
-
-               Next_Entity (Partial_View);
-            end loop;
+            Partial_View := Find_Partial_View (T);
 
             --  If the partial view was not found then the source code has
             --  errors and the transformation is not needed.
index b77b538..f098760 100644 (file)
@@ -2704,8 +2704,17 @@ package body Sem_Type is
          if Present (Full_View (Target_Typ)) then
             Target_Typ := Full_View (Target_Typ);
          else
-            pragma Assert (Present (Non_Limited_View (Target_Typ)));
-            Target_Typ := Non_Limited_View (Target_Typ);
+            --  In a spec expression or in an expression function, the use of
+            --  an incomplete type is legal; legality of the conversion will be
+            --  checked at freeze point of related entity.
+
+            if In_Spec_Expression then
+               return True;
+
+            else
+               pragma Assert (Present (Non_Limited_View (Target_Typ)));
+               Target_Typ := Non_Limited_View (Target_Typ);
+            end if;
          end if;
 
          --  Protect the front end against previously detected errors