[Ada] Spell preanalysis, preanalyze correctly
authorBob Duff <duff@adacore.com>
Wed, 30 May 2018 08:57:44 +0000 (08:57 +0000)
committerPierre-Marie de Rodat <pmderodat@gcc.gnu.org>
Wed, 30 May 2018 08:57:44 +0000 (08:57 +0000)
2018-05-30  Bob Duff  <duff@adacore.com>

gcc/ada/

* aspects.ads, contracts.adb, exp_util.adb, expander.adb, expander.ads,
freeze.adb, inline.adb, lib-xref.adb, sem.ads, sem_aggr.adb,
sem_attr.adb, sem_ch13.adb, sem_ch3.adb, sem_ch5.adb, sem_ch6.adb,
sem_ch8.adb, sem_dim.adb, sem_elab.adb, sem_res.adb, sem_res.ads,
sinfo.ads: Spell preanalysis, preanalyze correctly.

From-SVN: r260937

22 files changed:
gcc/ada/ChangeLog
gcc/ada/aspects.ads
gcc/ada/contracts.adb
gcc/ada/exp_util.adb
gcc/ada/expander.adb
gcc/ada/expander.ads
gcc/ada/freeze.adb
gcc/ada/inline.adb
gcc/ada/lib-xref.adb
gcc/ada/sem.ads
gcc/ada/sem_aggr.adb
gcc/ada/sem_attr.adb
gcc/ada/sem_ch13.adb
gcc/ada/sem_ch3.adb
gcc/ada/sem_ch5.adb
gcc/ada/sem_ch6.adb
gcc/ada/sem_ch8.adb
gcc/ada/sem_dim.adb
gcc/ada/sem_elab.adb
gcc/ada/sem_res.adb
gcc/ada/sem_res.ads
gcc/ada/sinfo.ads

index 330a11e7ab64d00b93b4a5493011a3179a86da9b..1e1056b6fd9a921118482a3c04cb0e41f59b3ce4 100644 (file)
@@ -1,3 +1,11 @@
+2018-05-30  Bob Duff  <duff@adacore.com>
+
+       * aspects.ads, contracts.adb, exp_util.adb, expander.adb, expander.ads,
+       freeze.adb, inline.adb, lib-xref.adb, sem.ads, sem_aggr.adb,
+       sem_attr.adb, sem_ch13.adb, sem_ch3.adb, sem_ch5.adb, sem_ch6.adb,
+       sem_ch8.adb, sem_dim.adb, sem_elab.adb, sem_res.adb, sem_res.ads,
+       sinfo.ads: Spell preanalysis, preanalyze correctly.
+
 2018-05-30  Bob Duff  <duff@adacore.com>
 
        * libgnat/g-sestin.ads: Rework documentation comments.
index 02439aa46c90d526ba5ac824b849110878100c95..16dcab8c9ef7ce921a8ec6a911ccb1e7eec4a67a 100644 (file)
@@ -59,7 +59,7 @@
 --    5. If the semantic analysis of expressions/names in the aspect should not
 --       occur at the point the aspect is defined, add code in the adequate
 --       semantic analysis procedure for the aspect. For example, this is the
---       case for aspects Pre and Post on subprograms, which are pre-analyzed
+--       case for aspects Pre and Post on subprograms, which are preanalyzed
 --       at the end of the declaration list to which the subprogram belongs,
 --       and fully analyzed (possibly with expansion) during the semantic
 --       analysis of subprogram bodies.
index e9d4c0fe29192637c17b8f0d939730a043b3f5aa..51cde068db15c5746409a79194ad8efee31729b2 100644 (file)
@@ -405,7 +405,7 @@ package body Contracts is
          then
             Analyze_Task_Contract (Defining_Entity (Decl));
 
-         --  For type declarations, we need to do the pre-analysis of Iterable
+         --  For type declarations, we need to do the preanalysis of Iterable
          --  aspect specifications.
 
          --  Other type aspects need to be resolved here???
index 0130307ae033b25ad9b72404f177030fc3df14c4..7573121c1547b793ac5f132af96fd8c1c363f9ae 100644 (file)
@@ -11194,7 +11194,7 @@ package body Exp_Util is
       --  Handle cases in which there is nothing to do. In GNATprove mode,
       --  removal of side effects is useful for the light expansion of
       --  renamings. This removal should only occur when not inside a
-      --  generic and not doing a pre-analysis.
+      --  generic and not doing a preanalysis.
 
       if not Expander_Active
         and (Inside_A_Generic or not Full_Analysis or not GNATprove_Mode)
index 2e552cdb8fae44adbe094c3145e88df4e4b1f860..f6d91dc0bb3df8a512f65bae6e086b33d5c7efaf 100644 (file)
@@ -125,12 +125,12 @@ package body Expander is
 
       --  The first is when are not generating code. In this mode the
       --  Full_Analysis flag indicates whether we are performing a complete
-      --  analysis, in which case Full_Analysis = True or a pre-analysis in
+      --  analysis, in which case Full_Analysis = True or a preanalysis in
       --  which case Full_Analysis = False. See the spec of Sem for more info
       --  on this.
 
       --  The second reason for the Expander_Active flag to be False is that
-      --  we are performing a pre-analysis. During pre-analysis all expansion
+      --  we are performing a preanalysis. During preanalysis all expansion
       --  activity is turned off to make sure nodes are semantically decorated
       --  but no extra nodes are generated. This is for instance needed for
       --  the first pass of aggregate semantic processing. Note that in this
index 5147cbe4e7817ef6f21d17450dd1a8167dc61f47..b3e933c59abd6811b851009bd3d9e64aa980d8f9 100644 (file)
@@ -135,12 +135,12 @@ package Expander is
    --        or declaration (see routines Start_Generic_Processing and
    --        End_Generic_Processing in Sem_Ch12)
    --
-   --    (b) when starting/completing the pre-analysis of an expression
-   --        (see the spec of package Sem for more info on pre-analysis.)
+   --    (b) when starting/completing the preanalysis of an expression
+   --        (see the spec of package Sem for more info on preanalysis.)
    --
    --  Note that when processing a spec expression (In_Spec_Expression
    --  is True) or performing semantic analysis of a generic spec or body
-   --  (Inside_A_Generic) or when performing pre-analysis (Full_Analysis is
+   --  (Inside_A_Generic) or when performing preanalysis (Full_Analysis is
    --  False) the Expander_Active flag is False.
 
    procedure Expand (N : Node_Id);
index 72b8f9536b2ad19406ea7f6cadc0da74659c5177..598714980b6ab5325dc58bb4856e5d31edbfe413 100644 (file)
@@ -186,7 +186,7 @@ package body Freeze is
    --  This procedure is called for each subprogram to complete processing of
    --  default expressions at the point where all types are known to be frozen.
    --  The expressions must be analyzed in full, to make sure that all error
-   --  processing is done (they have only been pre-analyzed). If the expression
+   --  processing is done (they have only been preanalyzed). If the expression
    --  is not an entity or literal, its analysis may generate code which must
    --  not be executed. In that case we build a function body to hold that
    --  code. This wrapper function serves no other purpose (it used to be
index 72cb5c00ac259e9edf2528fef9504173c03fbd9e..336b336365af134d0db176effc333e3efcc279dd 100644 (file)
@@ -1156,7 +1156,7 @@ package body Inline is
          Append (Body_To_Analyze, Declarations (N));
       end if;
 
-      --  The body to inline is pre-analyzed. In GNATprove mode we must disable
+      --  The body to inline is preanalyzed. In GNATprove mode we must disable
       --  full analysis as well so that light expansion does not take place
       --  either, and name resolution is unaffected.
 
index 8eb6504e7007c87b4d8bd05f0aa3fab5c07d1564..eaa7aa64c4945be21f4644c30c6dacbeec54b925 100644 (file)
@@ -666,7 +666,7 @@ package body Lib.Xref is
 
       --  Do not generate references if we are within a postcondition sub-
       --  program, because the reference does not comes from source, and the
-      --  pre-analysis of the aspect has already created an entry for the ALI
+      --  preanalysis of the aspect has already created an entry for the ALI
       --  file at the proper source location.
 
       if Chars (Current_Scope) = Name_uPostconditions then
index da9a593cc8ce2a9fa9ed1847409eebab076686cf..0d8f41d7a074ee6f503b69c4cdb0c6f1662cced3 100644 (file)
 --  code for the pragma is generated.
 
 ------------------
--- Pre-Analysis --
+-- Preanalysis --
 ------------------
 
 --  For certain kind of expressions, such as aggregates, we need to defer
 --  expanded code for "new Thing (Function_Call)".
 
 --  To remedy this situation we introduce a flag that signals whether we want a
---  full analysis (i.e. expansion is enabled) or a pre-analysis which performs
+--  full analysis (i.e. expansion is enabled) or a preanalysis which performs
 --  Analysis and Resolution but no expansion.
 
---  After the complete pre-analysis of an expression has been carried out we
+--  After the complete preanalysis of an expression has been carried out we
 --  can transform the expression and then carry out the full three stage
 --  (Analyze-Resolve-Expand) cycle on the transformed expression top-down so
 --  that the expansion of inner expressions happens inside the newly generated
 --  node for the parent expression.
 
 --  Note that the difference between processing of default expressions and
---  pre-analysis of other expressions is that we do carry out freezing in
+--  preanalysis of other expressions is that we do carry out freezing in
 --  the latter but not in the former (except for static scalar expressions).
 --  The routine that performs preanalysis and corresponding resolution is
 --  called Preanalyze_And_Resolve and is in Sem_Res.
@@ -214,12 +214,12 @@ package Sem is
    -----------------------------
 
    Full_Analysis : Boolean := True;
-   --  Switch to indicate if we are doing a full analysis or a pre-analysis.
+   --  Switch to indicate if we are doing a full analysis or a preanalysis.
    --  In normal analysis mode (Analysis-Expansion for instructions or
    --  declarations) or (Analysis-Resolution-Expansion for expressions) this
    --  flag is set. Note that if we are not generating code the expansion phase
    --  merely sets the Analyzed flag to True in this case. If we are in
-   --  Pre-Analysis mode (see above) this flag is set to False then the
+   --  Preanalysis mode (see above) this flag is set to False then the
    --  expansion phase is skipped.
    --
    --  When this flag is False the flag Expander_Active is also False (the
@@ -688,9 +688,9 @@ package Sem is
    --  Determine whether preanalysis is active at the point of invocation
 
    procedure Preanalyze (N : Node_Id);
-   --  Performs a pre-analysis of node N. During pre-analysis no expansion is
-   --  carried out for N or its children. For more info on pre-analysis read
-   --  the spec of Sem.
+   --  Performs a preanalysis of node N. During preanalysis no expansion is
+   --  carried out for N or its children. See above for more info on
+   --  preanalysis.
 
    generic
       with procedure Action (Item : Node_Id);
index 5eaf4622d80bd32379891d6f1acc712a8baa8e7c..5e8705f90af000d51c3a119f2bac37cdf04a11cb 100644 (file)
@@ -115,7 +115,7 @@ package body Sem_Aggr is
    --  expressions allowed for a limited component association (namely, an
    --  aggregate, function call, or <> notation). Report error for violations.
    --  Expression is also OK in an instance or inlining context, because we
-   --  have already pre-analyzed and it is known to be type correct.
+   --  have already preanalyzed and it is known to be type correct.
 
    procedure Check_Qualified_Aggregate (Level : Nat; Expr : Node_Id);
    --  Given aggregate Expr, check that sub-aggregates of Expr that are nested
index f94cbadc9f813c4f0fb3260eb5e71bc6bc779ff2..cb1b2d5d2cc71acfad1de5690238244619380557 100644 (file)
@@ -1799,7 +1799,7 @@ package body Sem_Attr is
             --  designated type of the access type, since the type of the
             --  referenced array is this type (see AI95-00106).
 
-            --  As done elsewhere, freezing must not happen when pre-analyzing
+            --  As done elsewhere, freezing must not happen when preanalyzing
             --  a pre- or postcondition or a default value for an object or for
             --  a formal parameter.
 
index d9a0344f4069bf948d6bf18276a52a8ed8397adb..d287f2632b53b62f2c52872cdb3831c65aac2b9e 100644 (file)
@@ -119,7 +119,7 @@ package body Sem_Ch13 is
    --  Build the declaration for a predicate function. The declaration is built
    --  at the end of the declarative part containing the type definition, which
    --  may be before the freeze point of the type. The predicate expression is
-   --  pre-analyzed at this point, to catch visibility errors.
+   --  preanalyzed at this point, to catch visibility errors.
 
    procedure Build_Predicate_Functions (Typ : Entity_Id; N : Node_Id);
    --  If Typ has predicates (indicated by Has_Predicates being set for Typ),
@@ -8839,7 +8839,7 @@ package body Sem_Ch13 is
 
             --  within a generic unit, prevent a double analysis of the body
             --  which will not be marked analyzed yet. This will happen when
-            --  the freeze node is created during the pre-analysis of an
+            --  the freeze node is created during the preanalysis of an
             --  expression function.
 
             if Inside_A_Generic then
index 3bb406da68a1fb6cdadaed2cbb51533f87995b65..b9186d73ae347509f148b03ce2f0cf915bde0ccd 100644 (file)
@@ -2218,7 +2218,7 @@ package body Sem_Ch3 is
       --  Context denotes the owner of the declarative list.
 
       procedure Check_Entry_Contracts;
-      --  Perform a pre-analysis of the pre- and postconditions of an entry
+      --  Perform a preanalysis of the pre- and postconditions of an entry
       --  declaration. This must be done before full resolution and creation
       --  of the parameter block, etc. to catch illegal uses within the
       --  contract expression. Full analysis of the expression is done when
@@ -5286,7 +5286,7 @@ package body Sem_Ch3 is
 
       --  Finally this happens in some complex cases when validity checks are
       --  enabled, where the same subtype declaration may be analyzed twice.
-      --  This can happen if the subtype is created by the pre-analysis of
+      --  This can happen if the subtype is created by the preanalysis of
       --  an attribute tht gives the range of a loop statement, and the loop
       --  itself appears within an if_statement that will be rewritten during
       --  expansion.
index 7df4fa99d13fbc419f44ecfa4d1cbc5b8448e3aa..3ce57ea3d4b38eff5535d24bb199ac21bb4c0d64 100644 (file)
@@ -2278,7 +2278,7 @@ package body Sem_Ch5 is
          begin
 
             --  If the domain of iteration is an array component that depends
-            --  on a discriminant, create actual subtype for it. Pre-analysis
+            --  on a discriminant, create actual subtype for it. preanalysis
             --  does not generate the actual subtype of a selected component.
 
             if Nkind (Iter_Name) = N_Selected_Component
@@ -2702,7 +2702,7 @@ package body Sem_Ch5 is
       --  If the iteration is given by a range, create temporaries and
       --  assignment statements block to capture the bounds and perform
       --  required finalization actions in case a bound includes a function
-      --  call that uses the temporary stack. We first pre-analyze a copy of
+      --  call that uses the temporary stack. We first preanalyze a copy of
       --  the range in order to determine the expected type, and analyze and
       --  resolve the original bounds.
 
@@ -3131,7 +3131,7 @@ package body Sem_Ch5 is
 
          else
             --  A quantified expression that appears in a pre/post condition
-            --  is pre-analyzed several times.  If the range is given by an
+            --  is preanalyzed several times.  If the range is given by an
             --  attribute reference it is rewritten as a range, and this is
             --  done even with expansion disabled. If the type is already set
             --  do not reanalyze, because a range with static bounds may be
index 1580be1a2ad6116dbf651eb2d0bc8d0fe4b4c49c..f000f71a3a9a6bbc4dbb67d98d5d95ede33dd2c0 100644 (file)
@@ -708,7 +708,7 @@ package body Sem_Ch6 is
          Set_Corresponding_Body (N, Defining_Entity (New_Body));
          Set_Corresponding_Spec (New_Body, Def_Id);
 
-         --  Within a generic pre-analyze the original expression for name
+         --  Within a generic preanalyze the original expression for name
          --  capture. The body is also generated but plays no role in
          --  this because it is not part of the original source.
 
@@ -3643,7 +3643,7 @@ package body Sem_Ch6 is
 
                if No (Spec_Id) and then GNATprove_Mode
 
-                 --  Inlining does not apply during pre-analysis of code
+                 --  Inlining does not apply during preanalysis of code
 
                  and then Full_Analysis
 
index 1716cc476669e9e23b1212bca703051f15074026..02471d73bbb368ad00d43a8a4eb61d4f1c1eeaec 100644 (file)
@@ -7109,7 +7109,7 @@ package body Sem_Ch8 is
             end if;
 
          --  If the selected component appears within a default expression
-         --  and it has an actual subtype, the pre-analysis has not yet
+         --  and it has an actual subtype, the preanalysis has not yet
          --  completed its analysis, because Insert_Actions is disabled in
          --  that context. Within the init proc of the enclosing type we
          --  must complete this analysis, if an actual subtype was created.
index 9cdc9bb68700f842f15ff7db899d52d7200fe3a7..884c86f5aa2011cd6d5da9aa097de1b024cba2c1 100644 (file)
@@ -1558,7 +1558,7 @@ package body Sem_Dim is
             --  For relational operations, only dimension checking is
             --  performed (no propagation). If one operand is the result
             --  of constant folding the dimensions may have been lost
-            --  in a tree copy, so assume that pre-analysis has verified
+            --  in a tree copy, so assume that preanalysis has verified
             --  that dimensions are correct.
 
             elsif N_Kind in N_Op_Compare then
index e3169952803cd52bb7ca0245cd0c16726f4d8578..72bb0cbc46e02bec6c7e821a50d72d05c5a39500 100644 (file)
@@ -12945,7 +12945,7 @@ package body Sem_Elab is
       then
          return;
 
-      --  Nothing to do if call is being pre-analyzed, as when within a
+      --  Nothing to do if call is being preanalyzed, as when within a
       --  pre/postcondition, a predicate, or an invariant.
 
       elsif In_Spec_Expression then
index a2788d99922c6c02ae5c374f0b557df25aff1cd1..9df0cd6ae5a000f7d8edcdfbef6eb6a748e36994 100644 (file)
@@ -6676,7 +6676,7 @@ package body Sem_Res is
                Cannot_Inline
                  ("cannot inline & (in default expression)?", N, Nam_UA);
 
-            --  Inlining should not be performed during pre-analysis
+            --  Inlining should not be performed during preanalysis
 
             elsif Full_Analysis then
 
index 01aad880459192aa895301ee5c2c124a58dc50a3..58c8b5ebaa6515a511a1eb1e431fb3912c9e21a3 100644 (file)
@@ -85,9 +85,9 @@ package Sem_Res is
    --  The parameter T is the Typ for the corresponding resolve call.
 
    procedure Preanalyze_And_Resolve (N : Node_Id; T : Entity_Id);
-   --  Performs a pre-analysis of expression node N. During pre-analysis, N is
+   --  Performs a preanalysis of expression node N. During preanalysis, N is
    --  analyzed and then resolved against type T, but no expansion is carried
-   --  out for N or its children. For more info on pre-analysis read the spec
+   --  out for N or its children. For more info on preanalysis read the spec
    --  of Sem.
 
    procedure Preanalyze_And_Resolve (N : Node_Id);
index b09ca100044cd37a83885f2f99749eec9f926103..6aaeff86c2594bdad5f23db04ead32baa524dcb6 100644 (file)
@@ -640,7 +640,7 @@ package Sinfo is
 
    --  The tree after this light expansion should be fully analyzed
    --  semantically, which sometimes requires the insertion of semantic
-   --  pre-analysis, for example for subprogram contracts and pragma
+   --  preanalysis, for example for subprogram contracts and pragma
    --  check/assert. In particular, all expression must have their proper type,
    --  and semantic links should be set between tree nodes (partial to full
    --  view, etc.) Some kinds of nodes should be either absent, or can be