[Ada] Avoid spurious error in GNATprove mode on non-null access types
[gcc.git] / gcc / ada / ChangeLog
index f01e411b037bf19725f8fa0b0a5755362e9f4037..e422ee771da008e0733174191e787eaf76572926 100644 (file)
@@ -1,3 +1,330 @@
+2019-08-21  Yannick Moy  <moy@adacore.com>
+
+       * checks.adb (Install_Null_Excluding_Check): Do not install
+       check in GNATprove mode.
+
+2019-08-21  Yannick Moy  <moy@adacore.com>
+
+       * sem_spark.adb (Process_Path): Do nothing on address of
+       subprogram.
+
+2019-08-21  Eric Botcazou  <ebotcazou@adacore.com>
+
+       * exp_util.adb (Finalize_Address): Deal consistently with
+       subtypes of private protected types.
+
+2019-08-21  Piotr Trojanek  <trojanek@adacore.com>
+
+       * exp_util.adb (Corresponding_Runtime_Package): Use high-level
+       Is_Protected_Type.
+       * sem_ch8.adb (Analyze_Renamed_Entry): Likewise.
+       * sem_ch9.adb (Analyze_Requeue): Likewise.
+
+2019-08-21  Javier Miranda  <miranda@adacore.com>
+
+       * sem_util.adb (Update_Named_Associations): Update
+       First_Named_Actual when the subprogram call has a single named
+       actual.
+
+2019-08-21  Joel Brobecker  <brobecker@adacore.com>
+
+       * doc/Makefile (mk_empty_dirs): New (PHONY) rule.
+       (%.html, %.pdf, %.txt, %.info, %.texinfo): Add dependencies on
+       mk_empty_dirs.
+
+2019-08-21  Justin Squirek  <squirek@adacore.com>
+
+       * doc/gnat_rm/implementation_defined_pragmas.rst: Modify
+       documentation to reflect expected behavior.
+       * gnat_rm.texi: Regenerate.
+       * sem_prag.adb (Analyze_Pragma): Modify handling of pragma
+       Max_Entry_Queue_Length to not reject integer values of negative
+       one.
+       * sem_util.adb (Get_Max_Queue_Length): Add processing for values
+       of negative one to fit within the current scheme.
+
+2019-08-20  Arnaud Charlet  <charlet@adacore.com>
+
+       * exp_unst.adb (Unnest_Subprograms, Unnest_Subprogram): Take
+       inlining (-gnatn) into account.
+
+2019-08-20  Ed Schonberg  <schonberg@adacore.com>
+
+       * exp_ch13.adb (Expand_N_Attribute_Definition_Clause, case
+       Storage_Size): If the expression for Storage_Size is not static
+       it may depend on characterstics of another type that may bot be
+       frozen yet, so the elaboration of the expression for the aspect
+       must be attached directly to the freeze actions of the type to
+       which it applies.
+
+2019-08-20  Piotr Trojanek  <trojanek@adacore.com>
+
+       * exp_util.adb (Build_DIC_Procedure_Declaration): Set the last
+       entity of the generated Default_Initial_Condition procedure in
+       order to construct a proper entity chain.
+
+2019-08-20  Yannick Moy  <moy@adacore.com>
+
+       * exp_spark.adb (Expand_SPARK_N_Slice_Or_Indexed_Component):
+       Renaming of function to apply to slices as well.
+       (Expand_SPARK): Expand prefix of slices of access type.
+
+2019-08-20  Bob Duff  <duff@adacore.com>
+
+       * exp_aggr.adb (Expand_Array_Aggregate): Use build-in-place in
+       the nonlimited case in STEP 4. This improves the efficiency of
+       things like (1 .. 1000 => <>).  We still generate some code for
+       that, unfortunately, but it is much improved.
+       (Aggr_Assignment_OK_For_Backend): Return false if <> components
+       are present.
+
+2019-08-20  Bob Duff  <duff@adacore.com>
+
+       * exp_ch6.adb (Needs_BIP_Alloc_Form): Call
+       Requires_Transient_Scope rather than checking constrainedness
+       and so forth.  We have previously improved
+       Requires_Transient_Scope to return False in various cases,
+       notably a limited record with an access discriminant. This
+       change takes advantage of that to avoid using the secondary
+       stack for functions returning such types.
+       (Make_Build_In_Place_Call_In_Allocator): Be consistent by
+       calling Needs_BIP_Alloc_Form rather than Is_Constrained and so
+       forth.
+       * sem_ch4.adb (Analyze_Allocator): The above change causes the
+       compiler to generate code that is not legal Ada, in particular
+       an uninitialized allocator for indefinite subtype.  This is
+       harmless, so we suppress the error message in this case.
+
+2019-08-20  Gary Dismukes  <dismukes@adacore.com>
+
+       * ali.adb, ali.ads, aspects.adb, checks.ads, checks.adb,
+       doc/gnat_rm/implementation_defined_pragmas.rst,
+       doc/gnat_ugn/building_executable_programs_with_gnat.rst,
+       einfo.ads, exp_aggr.adb, exp_ch11.adb, exp_ch3.ads, exp_ch4.adb,
+       exp_disp.adb, inline.adb, libgnat/a-locale.ads,
+       libgnat/s-soflin.ads, par_sco.adb, repinfo.adb, sem_ch5.adb,
+       sem_disp.adb, sem_elab.adb, sem_eval.adb, sem_spark.adb,
+       sem_spark.ads, sinfo.ads: Minor reformattings, typo fixes and
+       and rewordings.
+
+2019-08-20  Dmitriy Anisimkov  <anisimko@adacore.com>
+
+       * adaint.c (__gnat_to_gm_time): On _WIN32, don't round time up
+       to even second.
+
+2019-08-20  Ed Schonberg  <schonberg@adacore.com>
+
+       * sem_ch3.adb (Analyze_Object_Declaration): If actual type is
+       private and distinct from nominal type in declaration, propagate
+       flags Is_Constr_Subt_For_U_Nominal and _UN_Aliased to full view
+       of private type.
+
+2019-08-20  Ed Schonberg  <schonberg@adacore.com>
+
+       * exp_attr.adb (Expand_Loop_Entry_Attribute): When expanding a
+       loop entry attribute for a while_loop we construct a function
+       that incorporates the expanded condition of the loop. The itypes
+       that may be generated in that expansion must carry the scope of
+       the constructed function for proper handling in gigi.
+
+2019-08-20  Ed Schonberg  <schonberg@adacore.com>
+
+       * exp_disp.adb (Build_Class_Wide_Check, Replace_Formals): When a
+       dispatching call tp a subprogram with a class-wide precondition
+       occurrs in the same declarative part as the ancestor subprogram
+       being called, the`expression for the precondition has not been
+       analyzed yet. Such a call may appear, e.g. in an expression
+       function. In that case, the replacement of formals by actuals in
+       the call cannot use the formal entities of the subprogram being
+       called, and the occurrence of the formals in the expression must
+       be located by name (Chars fields) as would be done at a later
+       freeze point, when the expression is resolved in the context of
+       the subprogram itself.
+
+2019-08-20  Bob Duff  <duff@adacore.com>
+
+       * sem_prag.adb (Persistent_BSS): If an initialization is present
+       because of Initialize_Scalars or Normalize_Scalars, generate an
+       implicit pragma Suppress_Initialization to remove that, because
+       initialization is not allowed for these variables. Other
+       initializations remain illegal.
+
+2019-08-20  Gary Dismukes  <dismukes@adacore.com>
+
+       * sem_ch3.adb (OK_For_Limited_Init_In_05): In the case of type
+       conversions, apply the recursive call to the Original_Node of
+       the expression Exp rather than the Expression of the
+       Original_Node, in the case where Exp has been rewritten;
+       otherwise, when Original_Node is the same as Exp, apply the
+       recursive call to the Expression.
+       (Check_Initialization): Revise condition for special check on
+       type conversions of limited function calls to test Original_Node
+       (avoiding spurious errors on expanded unchecked conversions
+       applied to build-in-place dispatching calls).
+
+2019-08-20  Patrick Bernardi  <bernardi@adacore.com>
+
+       * exp_aggr.adb (Expand_Record_Aggregate): Always convert a
+       record Aggregate to assignment statements if the option
+       Aggregate_Individually_Assign is set.
+       * opt.ads (Aggregate_Individually_Assign): New option.
+       * par-prag.adb (Prag): Add Pragma_Aggregate_Individually_Assign.
+       * sem_prag.adb (Analyze_Pragma): Likewise.
+       * snames.ads-tmpl: Add Pragma_Aggregate_Individually_Assign and
+       Name_Aggregate_Individually_Assign.
+       * doc/gnat_rm/implementation_defined_pragmas.rst: Document
+       pragma Aggregate_Individually_Assign.
+       * gnat_rm.texi: Regenerate.
+
+2019-08-20  Bob Duff  <duff@adacore.com>
+
+       * par-ch4.adb: Minor wording change in error messages.
+       * sem_aggr.adb (Resolve_Delta_Aggregate): Emit an error for Ada
+       versions prior to Ada 2020.
+
+2019-08-20  Bob Duff  <duff@adacore.com>
+
+       * freeze.adb (Is_Atomic_VFA_Aggregate): Make the temp for
+       initialization of the atomic variable be constant. This is
+       cleaner, and might improve efficiency.
+
+2019-08-20  Eric Botcazou  <ebotcazou@adacore.com>
+
+       * repinfo.adb (List_Record_Info): In -gnatR4 mode, set the
+       relevant flag on the implicit base type of a record subtype.
+
+2019-08-20  Bob Duff  <duff@adacore.com>
+
+       * sem_eval.adb (Expr_Value): Implement the case of an unchecked
+       conversion of a static expression.
+
+2019-08-20  Bob Duff  <duff@adacore.com>
+
+       * sem_ch13.adb (Is_Null_Array): New function, used to detect the
+       null array case; used to warn about uncheckedly converting to a
+       zero-sized array.  It is unfortunate that we can't just check
+       the size, and warn on all cases of converting from a
+       nonzero-sized type to a zero-sized one. That's because "0" means
+       two different things: "size is zero" and "size is unknown".
+       Until we fix that design flaw, we need this more targeted fix.
+
+2019-08-20  Bob Duff  <duff@adacore.com>
+
+       * libgnat/a-cborma.adb, libgnat/a-cborse.adb (Clear): Repeatedly
+       call Delete. This avoids clearing the free list, which
+       substantially speeds up future Delete operations.
+
+2019-08-20  Bob Duff  <duff@adacore.com>
+
+       * sem_ch13.adb (Component_Order_Check): New procedure to check
+       for out-of-order clauses.
+       * warnsw.ads, warnsw.adb: New -gnatw_r switch.
+       * doc/gnat_ugn/building_executable_programs_with_gnat.rst:
+       Document new switch.
+       * gnat_ugn.texi: Regenerate.
+
+2019-08-20  Bob Duff  <duff@adacore.com>
+
+       * sem_ch13.adb (Object_Size): Give an error for zero. It really
+       rubs me the wrong way that we don't honor "for T'Object_Size use
+       0;", but it's not important enough to fix. In any case, if we're
+       not going to obey the clause, we should give an error.
+
+2019-08-20  Bob Duff  <duff@adacore.com>
+
+       * errout.adb (Error_Msg_Internal): Set Warn_Err in case of
+       Is_Style_Msg.
+       * erroutc.adb (Output_Msg_Text): Do Warnings_Treated_As_Errors
+       processing and [warning-as-error] modification for style
+       messages.  Clean up code, simplify, remove unnecessary block
+       statement, add renaming of table entry.
+       * erroutc.ads (Warning_Treated_As_Error): Fix comment: no such
+       thing as Set_Warning_As_Error.
+       * opt.ads: Clean up comments and move related declarations near
+       each other.
+       * par-prag.adb: Process Warning_As_Error. This is necessary
+       because many style warning happen during parsing.
+       * sem_prag.adb: Use new Acquire_Warning_Match_String.
+       * sem_util.ads, sem_util.adb (Acquire_Warning_Match_String): New
+       function shared by par-prag.adb and sem_prag.adb. Replaces the
+       procedure in sem_prag.adb. Avoid use of global variables.
+       * stringt.ads, stringt.adb (To_String): New function to convert
+       String_Id to String.
+       * doc/gnat_rm/implementation_defined_pragmas.rst: Document the
+       new feature.
+       * gnat_rm.texi: Regenerate.
+
+2019-08-20  Eric Botcazou  <ebotcazou@adacore.com>
+
+       * lib.ads: Add with clause for GNAT.HTable.
+       Add pragma Inline for Is_Loaded and alphabetize the list.
+       (Unit_Name_Table_Size): New constant.
+       (Unit_Name_Header_Num): New subtype.
+       (Unit_Name_Hash): New function declaration.
+       (Unit_Names): New simple hash table.
+       (Init_Unit_Name): New procedure declaration.
+       * lib.adb (Set_Unit_Name): Unregister the old name in the table,
+       if any, and then register the new name.
+       (Init_Unit_Name): New procedure.
+       (Is_Loaded): Reimplement using a lookup in the names table.
+       (Remove_Unit): Unregister the name.
+       (Unit_Name_Hash): New function.
+       * lib-load.adb (Create_Dummy_Package_Unit): Call Init_Unit_Name.
+       (Load_Unit): Use a lookup in the names table to find out whether
+       the unit has already been loaded.  Call Init_Unit_Name and then
+       Remove_Unit if the loading has failed.
+       (Make_Child_Decl_Unit): Call Init_Unit_Name.
+       (Make_Instance_Unit): Likewise.
+       * lib-writ.adb (Ensure_System_Dependency): Likewise.
+
+2019-08-20  Bob Duff  <duff@adacore.com>
+
+       * sem_ch13.adb (Record_Hole_Check): Initialize After_Last.
+
+2019-08-20  Piotr Trojanek  <trojanek@adacore.com>
+
+       * impunit.adb (Get_Kind_Of_Unit): Revert change for adapting
+       this routine for gnatprove.
+
+2019-08-20  Arnaud Charlet  <charlet@adacore.com>
+
+       * doc/gnat_rm/implementation_defined_pragmas.rst: Document
+       requirements for Pragma Lock_Free.
+       * gnat_rm.texi: Regenerate.
+
+2019-08-20  Philippe Gil  <gil@adacore.com>
+
+       * bindgen.adb (Gen_Main): Set gnat_argc/gnat_argv to argc/argv
+       only when still uninitialized.
+
+2019-08-20  Bob Duff  <duff@adacore.com>
+
+       * libgnat/g-comlin.ads (Exit_From_Command_Line): Fix
+       documentation for GNAT.Command_Line.
+
+2019-08-20  Pierre-Marie de Rodat  <derodat@adacore.com>
+
+       PR ada/91492
+       * gcc-interface/lang.opt (-fdump-scos): Add missing dot at the
+       end of the documentation.
+
+2019-08-20  Richard Sandiford  <richard.sandiford@arm.com>
+
+       * gcc-interface/misc.c (default_pass_by_ref): Update call to
+       pass_by_reference.
+
+2019-08-19  Bob Duff  <duff@adacore.com>
+
+       * doc/gnat_rm/implementation_advice.rst: Fix documentation for
+       stream oriented attributes.
+       * gnat_rm.texi: Regenerate.
+
+2019-08-19  Gary Dismukes  <dismukes@adacore.com>
+
+       * einfo.ads (E_Function, E_Procedure): Update comments to
+       reflect that Renamed_Entity is also used for nongeneric
+       subprograms.
+
 2019-08-19  Eric Botcazou  <ebotcazou@adacore.com>
 
        * sem_prag.adb (Is_Before_First_Decl): Deal with rewritten