OSDN Git Service

./:
[pf3gnuchains/gcc-fork.git] / gcc / ada / lib-xref.adb
index 014a9e9..ec47ff9 100644 (file)
@@ -6,7 +6,7 @@
 --                                                                          --
 --                                 B o d y                                  --
 --                                                                          --
---          Copyright (C) 1998-2003, Free Software Foundation, Inc.         --
+--          Copyright (C) 1998-2007, 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- --
@@ -16,8 +16,8 @@
 -- or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License --
 -- for  more details.  You should have  received  a copy of the GNU General --
 -- Public License  distributed with GNAT;  see file COPYING.  If not, write --
--- to  the Free Software Foundation,  59 Temple Place - Suite 330,  Boston, --
--- MA 02111-1307, USA.                                                      --
+-- to  the  Free Software Foundation,  51  Franklin  Street,  Fifth  Floor, --
+-- Boston, MA 02110-1301, USA.                                              --
 --                                                                          --
 -- GNAT was originally developed  by the GNAT team at  New York University. --
 -- Extensive contributions were provided by Ada Core Technologies Inc.      --
@@ -29,10 +29,14 @@ with Csets;    use Csets;
 with Elists;   use Elists;
 with Errout;   use Errout;
 with Lib.Util; use Lib.Util;
-with Namet;    use Namet;
 with Nlists;   use Nlists;
 with Opt;      use Opt;
+with Restrict; use Restrict;
+with Rident;   use Rident;
+with Sem;      use Sem;
 with Sem_Prag; use Sem_Prag;
+with Sem_Util; use Sem_Util;
+with Sem_Warn; use Sem_Warn;
 with Sinfo;    use Sinfo;
 with Sinput;   use Sinput;
 with Snames;   use Snames;
@@ -108,7 +112,13 @@ package body Lib.Xref is
 
          --  Definition must come from source
 
-         and then Comes_From_Source (E)
+         --  We make an exception for subprogram child units that have no
+         --  spec. For these we generate a subprogram declaration for library
+         --  use, and the corresponding entity does not come from source.
+         --  Nevertheless, all references will be attached to it and we have
+         --  to treat is as coming from user code.
+
+         and then (Comes_From_Source (E) or else Is_Child_Unit (E))
 
          --  And must have a reasonable source location that is not
          --  within an instance (all entities in instances are ignored)
@@ -126,10 +136,16 @@ package body Lib.Xref is
          Loc  := Original_Location (Sloc (E));
 
          Xrefs.Table (Indx).Ent := E;
+         Xrefs.Table (Indx).Def := No_Location;
          Xrefs.Table (Indx).Loc := No_Location;
+         Xrefs.Table (Indx).Typ := ' ';
          Xrefs.Table (Indx).Eun := Get_Source_Unit (Loc);
          Xrefs.Table (Indx).Lun := No_Unit;
          Set_Has_Xref_Entry (E);
+
+         if In_Inlined_Body then
+            Set_Referenced (E);
+         end if;
       end if;
    end Generate_Definition;
 
@@ -198,22 +214,132 @@ package body Lib.Xref is
       Def  : Source_Ptr;
       Ent  : Entity_Id;
 
+      function Is_On_LHS (Node : Node_Id) return Boolean;
+      --  Used to check if a node is on the left hand side of an assignment.
+      --  The following cases are handled:
+      --
+      --   Variable  Node is a direct descendant of an assignment statement.
+      --
+      --   Prefix    Of an indexed or selected component that is present in a
+      --             subtree rooted by an assignment statement. There is no
+      --             restriction of nesting of components, thus cases such as
+      --             A.B(C).D are handled properly.
+
+      ---------------
+      -- Is_On_LHS --
+      ---------------
+
+      --  Couldn't we use Is_Lvalue or whatever it is called ???
+
+      function Is_On_LHS (Node : Node_Id) return Boolean is
+         N : Node_Id := Node;
+
+      begin
+         --  Only identifiers are considered, is this necessary???
+
+         if Nkind (N) /= N_Identifier then
+            return False;
+         end if;
+
+         --  Reach the assignment statement subtree root. In the case of a
+         --  variable being a direct descendant of an assignment statement,
+         --  the loop is skiped.
+
+         while Nkind (Parent (N)) /= N_Assignment_Statement loop
+
+            --  Check whether the parent is a component and the
+            --  current node is its prefix.
+
+            if (Nkind (Parent (N)) = N_Selected_Component
+                  or else
+                Nkind (Parent (N)) = N_Indexed_Component)
+              and then Prefix (Parent (N)) = N
+            then
+               N := Parent (N);
+            else
+               return False;
+            end if;
+         end loop;
+
+         --  Parent (N) is assignment statement, check whether N is its name
+
+         return Name (Parent (N)) = N;
+      end Is_On_LHS;
+
+   --  Start of processing for Generate_Reference
+
    begin
       pragma Assert (Nkind (E) in N_Entity);
 
-      --  Never collect references if not in main source unit. However,
-      --  we omit this test if Typ is 'e' or 'k', since these entries are
-      --  really structural, and it is useful to have them in units
-      --  that reference packages as well as units that define packages.
-      --  We also omit the test for the case of 'p' since we want to
-      --  include inherited primitive operations from other packages.
-
-      if not In_Extended_Main_Source_Unit (N)
-        and then Typ /= 'e'
-        and then Typ /= 'p'
-        and then Typ /= 'k'
+      --  Check for obsolescent reference to package ASCII. GNAT treats this
+      --  element of annex J specially since in practice, programs make a lot
+      --  of use of this feature, so we don't include it in the set of features
+      --  diagnosed when Warn_On_Obsolescent_Features mode is set. However we
+      --  are required to note it as a violation of the RM defined restriction.
+
+      if E = Standard_ASCII then
+         Check_Restriction (No_Obsolescent_Features, N);
+      end if;
+
+      --  Check for reference to entity marked with Is_Obsolescent
+
+      --  Note that we always allow obsolescent references in the compiler
+      --  itself and the run time, since we assume that we know what we are
+      --  doing in such cases. For example the calls in Ada.Characters.Handling
+      --  to its own obsolescent subprograms are just fine.
+
+      --  In any case we do not generate warnings within the extended source
+      --  unit of the entity in question, since we assume the source unit
+      --  itself knows what is going on (and for sure we do not want silly
+      --  warnings, e.g. on the end line of an obsolescent procedure body).
+
+      if Is_Obsolescent (E)
+        and then not GNAT_Mode
+        and then not In_Extended_Main_Source_Unit (E)
       then
-         return;
+         Check_Restriction (No_Obsolescent_Features, N);
+
+         if Warn_On_Obsolescent_Feature then
+            Output_Obsolescent_Entity_Warnings (N, E);
+         end if;
+      end if;
+
+      --  Warn if reference to Ada 2005 entity not in Ada 2005 mode. We only
+      --  detect real explicit references (modifications and references).
+
+      if Comes_From_Source (N)
+        and then Is_Ada_2005_Only (E)
+        and then Ada_Version < Ada_05
+        and then Warn_On_Ada_2005_Compatibility
+        and then (Typ = 'm' or else Typ = 'r')
+      then
+         Error_Msg_NE ("& is only defined in Ada 2005?", N, E);
+      end if;
+
+      --  Never collect references if not in main source unit. However, we omit
+      --  this test if Typ is 'e' or 'k', since these entries are structural,
+      --  and it is useful to have them in units that reference packages as
+      --  well as units that define packages. We also omit the test for the
+      --  case of 'p' since we want to include inherited primitive operations
+      --  from other packages.
+
+      --  We also omit this test is this is a body reference for a subprogram
+      --  instantiation. In this case the reference is to the generic body,
+      --  which clearly need not be in the main unit containing the instance.
+      --  For the same reason we accept an implicit reference generated for
+      --  a default in an instance.
+
+      if not In_Extended_Main_Source_Unit (N) then
+         if Typ = 'e'
+           or else Typ = 'p'
+           or else Typ = 'i'
+           or else Typ = 'k'
+           or else (Typ = 'b' and then Is_Generic_Instance (E))
+         then
+            null;
+         else
+            return;
+         end if;
       end if;
 
       --  For reference type p, the entity must be in main source unit
@@ -222,32 +348,30 @@ package body Lib.Xref is
          return;
       end if;
 
-      --  Unless the reference is forced, we ignore references where
-      --  the reference itself does not come from Source.
+      --  Unless the reference is forced, we ignore references where the
+      --  reference itself does not come from Source.
 
       if not Force and then not Comes_From_Source (N) then
          return;
       end if;
 
-      --  Deal with setting entity as referenced, unless suppressed.
-      --  Note that we still do Set_Referenced on entities that do not
-      --  come from source. This situation arises when we have a source
-      --  reference to a derived operation, where the derived operation
-      --  itself does not come from source, but we still want to mark it
-      --  as referenced, since we really are referencing an entity in the
-      --  corresponding package (this avoids incorrect complaints that the
-      --  package contains no referenced entities).
+      --  Deal with setting entity as referenced, unless suppressed. Note that
+      --  we still do Set_Referenced on entities that do not come from source.
+      --  This situation arises when we have a source reference to a derived
+      --  operation, where the derived operation itself does not come from
+      --  source, but we still want to mark it as referenced, since we really
+      --  are referencing an entity in the corresponding package (this avoids
+      --  wrong complaints that the package contains no referenced entities).
 
       if Set_Ref then
 
-         --  For a variable that appears on the left side of an
-         --  assignment statement, we set the Referenced_As_LHS
-         --  flag since this is indeed a left hand side.
+         --  For a variable that appears on the left side of an assignment
+         --  statement, we set the Referenced_As_LHS flag since this is indeed
+         --  a left hand side. We also set the Referenced_As_LHS flag of a
+         --  prefix of selected or indexed component.
 
          if Ekind (E) = E_Variable
-           and then Nkind (Parent (N)) = N_Assignment_Statement
-           and then Name (Parent (N)) = N
-           and then No (Renamed_Object (E))
+           and then Is_On_LHS (N)
          then
             Set_Referenced_As_LHS (E);
 
@@ -257,11 +381,10 @@ package body Lib.Xref is
          elsif Is_Non_Significant_Pragma_Reference (N) then
             null;
 
-         --  A reference in an attribute definition clause does not
-         --  count as a reference except for the case of Address.
-         --  The reason that 'Address is an exception is that it
-         --  creates an alias through which the variable may be
-         --  referenced.
+         --  A reference in an attribute definition clause does not count as a
+         --  reference except for the case of Address. The reason that 'Address
+         --  is an exception is that it creates an alias through which the
+         --  variable may be referenced.
 
          elsif Nkind (Parent (N)) = N_Attribute_Definition_Clause
            and then Chars (Parent (N)) /= Name_Address
@@ -269,16 +392,43 @@ package body Lib.Xref is
          then
             null;
 
+         --  Constant completion does not count as a reference
+
+         elsif Typ = 'c'
+           and then Ekind (E) = E_Constant
+         then
+            null;
+
+         --  Record representation clause does not count as a reference
+
+         elsif Nkind (N) = N_Identifier
+           and then Nkind (Parent (N)) = N_Record_Representation_Clause
+         then
+            null;
+
+         --  Discriminants do not need to produce a reference to record type
+
+         elsif Typ = 'd'
+           and then Nkind (Parent (N)) = N_Discriminant_Specification
+         then
+            null;
+
          --  Any other occurrence counts as referencing the entity
 
          else
             Set_Referenced (E);
-         end if;
 
-         --  Check for pragma Unreferenced given
+            if Ekind (E) = E_Variable then
+               Set_Last_Assignment (E, Empty);
+            end if;
+         end if;
 
-         if Has_Pragma_Unreferenced (E) then
+         --  Check for pragma Unreferenced given and reference is within
+         --  this source unit (occasion for possible warning to be issued)
 
+         if Has_Pragma_Unreferenced (E)
+           and then In_Same_Extended_Unit (E, N)
+         then
             --  A reference as a named parameter in a call does not count
             --  as a violation of pragma Unreferenced for this purpose.
 
@@ -289,14 +439,35 @@ package body Lib.Xref is
                null;
 
             --  Neither does a reference to a variable on the left side
-            --  of an assignment
+            --  of an assignment.
 
-            elsif Ekind (E) = E_Variable
-              and then Nkind (Parent (N)) = N_Assignment_Statement
-              and then Name (Parent (N)) = N
-            then
+            elsif Is_On_LHS (N) then
                null;
 
+            --  For entry formals, we want to place the warning message on the
+            --  corresponding entity in the accept statement. The current scope
+            --  is the body of the accept, so we find the formal whose name
+            --  matches that of the entry formal (there is no link between the
+            --  two entities, and the one in the accept statement is only used
+            --  for conformance checking).
+
+            elsif Ekind (Scope (E)) = E_Entry then
+               declare
+                  BE : Entity_Id;
+
+               begin
+                  BE := First_Entity (Current_Scope);
+                  while Present (BE) loop
+                     if Chars (BE) = Chars (E) then
+                        Error_Msg_NE
+                          ("?pragma Unreferenced given for&", N, BE);
+                        exit;
+                     end if;
+
+                     Next_Entity (BE);
+                  end loop;
+               end;
+
             --  Here we issue the warning, since this is a real reference
 
             else
@@ -380,17 +551,22 @@ package body Lib.Xref is
            and then Present (Alias (E))
          then
             Ent := Alias (E);
-
-            loop
-               if Comes_From_Source (Ent) then
-                  exit;
-               elsif No (Alias (Ent)) then
+            while not Comes_From_Source (Ent) loop
+               if No (Alias (Ent)) then
                   return;
-               else
-                  Ent := Alias (Ent);
                end if;
+
+               Ent := Alias (Ent);
             end loop;
 
+         --  The internally created defining entity for a child subprogram
+         --  that has no previous spec has valid references.
+
+         elsif Is_Overloadable (E)
+           and then Is_Child_Unit (E)
+         then
+            Ent := E;
+
          --  Record components of discriminated subtypes or derived types
          --  must be treated as references to the original component.
 
@@ -415,7 +591,7 @@ package body Lib.Xref is
 
          Xrefs.Table (Indx).Loc := Ref;
 
-         --  Overriding operations are marked with 'P'.
+         --  Overriding operations are marked with 'P'
 
          if Typ = 'p'
            and then Is_Subprogram (N)
@@ -485,7 +661,6 @@ package body Lib.Xref is
 
    begin
       Formal := First_Entity (E);
-
       while Present (Formal) loop
          if Comes_From_Source (Formal) then
             Generate_Reference (E, Formal, 'z', False);
@@ -580,15 +755,25 @@ package body Lib.Xref is
 
                   elsif Is_Private_Type (Tref)
                     and then Present (Full_View (Tref))
-                    and then Is_Access_Type (Full_View (Tref))
                   then
-                     Tref := Directly_Designated_Type (Full_View (Tref));
-                     Left := '(';
-                     Right := ')';
+                     if Is_Access_Type (Full_View (Tref)) then
+                        Tref := Directly_Designated_Type (Full_View (Tref));
+                        Left := '(';
+                        Right := ')';
+
+                     --  If the full view is an array type, we also retrieve
+                     --  the corresponding component type, because the ali
+                     --  entry already indicates that this is an array.
+
+                     elsif Is_Array_Type (Full_View (Tref)) then
+                        Tref := Component_Type (Full_View (Tref));
+                        Left := '(';
+                        Right := ')';
+                     end if;
 
-                  --  If non-derived array, get component type.
-                  --  Skip component type for case of String
-                  --  or Wide_String, saves worthwhile space.
+                  --  If non-derived array, get component type. Skip component
+                  --  type for case of String or Wide_String, saves worthwhile
+                  --  space.
 
                   elsif Is_Array_Type (Tref)
                     and then Tref /= Standard_String
@@ -604,9 +789,7 @@ package body Lib.Xref is
                      exit;
                   end if;
 
-               --  For a subtype, go to ancestor subtype. If it is a
-               --  subtype created for a generic actual, not clear yet
-               --  what is the right type to use ???
+               --  For a subtype, go to ancestor subtype
 
                else
                   Tref := Ancestor_Subtype (Tref);
@@ -651,6 +834,19 @@ package body Lib.Xref is
             if Sloc (Tref) = Standard_Location
               or else Comes_From_Source (Tref)
             then
+               --  If the reference is a subtype created for a generic
+               --  actual, go to actual directly, the inner subtype is
+               --  not user visible.
+
+               if Nkind (Parent (Tref)) = N_Subtype_Declaration
+                 and then not Comes_From_Source (Parent (Tref))
+                 and then
+                  (Is_Wrapper_Package (Scope (Tref))
+                     or else Is_Generic_Instance (Scope (Tref)))
+               then
+                  Tref := First_Subtype (Base_Type (Tref));
+               end if;
+
                return;
             end if;
          end loop;
@@ -669,7 +865,10 @@ package body Lib.Xref is
       procedure Output_Import_Export_Info (Ent : Entity_Id) is
          Language_Name : Name_Id;
          Conv          : constant Convention_Id := Convention (Ent);
+
       begin
+         --  Generate language name from convention
+
          if Conv  = Convention_C then
             Language_Name := Name_C;
 
@@ -680,7 +879,7 @@ package body Lib.Xref is
             Language_Name := Name_Ada;
 
          else
-            --  These are the only languages that GPS knows about.
+            --  For the moment we ignore all other cases ???
 
             return;
          end if;
@@ -723,18 +922,18 @@ package body Lib.Xref is
       --  referenced in the main unit, which may mean that there is no xref
       --  entry for this entity yet in the list of references.
 
-      --  If we don't do something about this, we will end with an orphan
-      --  type reference, i.e. it will point to an entity that does not
-      --  appear within the generated references in the ali file. That is
-      --  not good for tools using the xref information.
+      --  If we don't do something about this, we will end with an orphan type
+      --  reference, i.e. it will point to an entity that does not appear
+      --  within the generated references in the ali file. That is not good for
+      --  tools using the xref information.
 
-      --  To fix this, we go through the references adding definition
-      --  entries for any unreferenced entities that can be referenced
-      --  in a type reference. There is a recursion problem here, and
-      --  that is dealt with by making sure that this traversal also
-      --  traverses any entries that get added by the traversal.
+      --  To fix this, we go through the references adding definition entries
+      --  for any unreferenced entities that can be referenced in a type
+      --  reference. There is a recursion problem here, and that is dealt with
+      --  by making sure that this traversal also traverses any entries that
+      --  get added by the traversal.
 
-      declare
+      Handle_Orphan_Type_References : declare
          J    : Nat;
          Tref : Entity_Id;
          L, R : Character;
@@ -742,10 +941,38 @@ package body Lib.Xref is
          Ent  : Entity_Id;
          Loc  : Source_Ptr;
 
+         procedure New_Entry (E : Entity_Id);
+         --  Make an additional entry into the Xref table for a type entity
+         --  that is related to the current entity (parent, type. ancestor,
+         --  progenitor, etc.).
+
+         ----------------
+         -- New_Entry --
+         ----------------
+
+         procedure New_Entry (E : Entity_Id) is
+         begin
+            if Present (E)
+              and then not Has_Xref_Entry (E)
+              and then Sloc (E) > No_Location
+            then
+               Xrefs.Increment_Last;
+               Indx := Xrefs.Last;
+               Loc  := Original_Location (Sloc (E));
+               Xrefs.Table (Indx).Ent := E;
+               Xrefs.Table (Indx).Loc := No_Location;
+               Xrefs.Table (Indx).Eun := Get_Source_Unit (Loc);
+               Xrefs.Table (Indx).Lun := No_Unit;
+               Set_Has_Xref_Entry (E);
+            end if;
+         end New_Entry;
+
+      --  Start of processing for Handle_Orphan_Type_References
+
       begin
          --  Note that this is not a for loop for a very good reason. The
-         --  processing of items in the table can add new items to the
-         --  table, and they must be processed as well
+         --  processing of items in the table can add new items to the table,
+         --  and they must be processed as well
 
          J := 1;
          while J <= Xrefs.Last loop
@@ -756,14 +983,25 @@ package body Lib.Xref is
               and then not Has_Xref_Entry (Tref)
               and then Sloc (Tref) > No_Location
             then
-               Xrefs.Increment_Last;
-               Indx := Xrefs.Last;
-               Loc  := Original_Location (Sloc (Tref));
-               Xrefs.Table (Indx).Ent := Tref;
-               Xrefs.Table (Indx).Loc := No_Location;
-               Xrefs.Table (Indx).Eun := Get_Source_Unit (Loc);
-               Xrefs.Table (Indx).Lun := No_Unit;
-               Set_Has_Xref_Entry (Tref);
+               New_Entry (Tref);
+
+               if Is_Record_Type (Ent)
+                 and then Present (Abstract_Interfaces (Ent))
+               then
+                  --  Add an entry for each one of the given interfaces
+                  --  implemented by type Ent.
+
+                  declare
+                     Elmt : Elmt_Id;
+
+                  begin
+                     Elmt := First_Elmt (Abstract_Interfaces (Ent));
+                     while Present (Elmt) loop
+                        New_Entry (Node (Elmt));
+                        Next_Elmt (Elmt);
+                     end loop;
+                  end;
+               end if;
             end if;
 
             --  Collect inherited primitive operations that may be
@@ -776,9 +1014,8 @@ package body Lib.Xref is
               and then Ent = Base_Type (Ent)
               and then In_Extended_Main_Source_Unit (Ent)
             then
-
                declare
-                  Op_List : Elist_Id := Primitive_Operations (Ent);
+                  Op_List : constant Elist_Id := Primitive_Operations (Ent);
                   Op      : Elmt_Id;
                   Prim    : Entity_Id;
 
@@ -787,11 +1024,10 @@ package body Lib.Xref is
                   --  through several derivations.
 
                   function Parent_Op (E : Entity_Id) return Entity_Id is
-                     Orig_Op : Entity_Id := Alias (E);
+                     Orig_Op : constant Entity_Id := Alias (E);
                   begin
                      if No (Orig_Op) then
                         return Empty;
-
                      elsif not Comes_From_Source (E)
                        and then not Has_Xref_Entry (Orig_Op)
                        and then Comes_From_Source (Orig_Op)
@@ -804,9 +1040,7 @@ package body Lib.Xref is
 
                begin
                   Op := First_Elmt (Op_List);
-
                   while Present (Op) loop
-
                      Prim := Parent_Op (Node (Op));
 
                      if Present (Prim) then
@@ -828,7 +1062,7 @@ package body Lib.Xref is
 
             J := J + 1;
          end loop;
-      end;
+      end Handle_Orphan_Type_References;
 
       --  Now we have all the references, including those for any embedded
       --  type references, so we can sort them, and output them.
@@ -949,6 +1183,8 @@ package body Lib.Xref is
          -- Name_Change --
          -----------------
 
+         --  Why a string comparison here??? Why not compare Name_Id values???
+
          function Name_Change (X : Entity_Id) return Boolean is
          begin
             Get_Unqualified_Name_String (Chars (X));
@@ -1033,11 +1269,100 @@ package body Lib.Xref is
                Right : Character;
                --  Used for {} or <> or () for type reference
 
+               procedure Check_Type_Reference
+                 (Ent : Entity_Id;
+                  List_Interface : Boolean);
+               --  Find whether there is a meaningful type reference for
+               --  Ent, and display it accordingly. If List_Interface is
+               --  true, then Ent is a progenitor interface of the current
+               --  type entity being listed. In that case list it as is,
+               --  without looking for a type reference for it.
+
                procedure Output_Instantiation_Refs (Loc : Source_Ptr);
                --  Recursive procedure to output instantiation references for
                --  the given source ptr in [file|line[...]] form. No output
                --  if the given location is not a generic template reference.
 
+               procedure Output_Overridden_Op (Old_E : Entity_Id);
+               --  For a subprogram that is overriding, display information
+               --  about the inherited operation that it overrides.
+
+               --------------------------
+               -- Check_Type_Reference --
+               --------------------------
+
+               procedure Check_Type_Reference
+                 (Ent : Entity_Id;
+                  List_Interface : Boolean)
+               is
+               begin
+                  if List_Interface then
+
+                     --  This is a progenitor interface of the type for
+                     --  which xref information is being generated.
+
+                     Tref  := Ent;
+                     Left  := '<';
+                     Right := '>';
+
+                  else
+                     Get_Type_Reference (Ent, Tref, Left, Right);
+                  end if;
+
+                  if Present (Tref) then
+
+                     --  Case of standard entity, output name
+
+                     if Sloc (Tref) = Standard_Location then
+                        Write_Info_Char (Left);
+                        Write_Info_Name (Chars (Tref));
+                        Write_Info_Char (Right);
+
+                     --  Case of source entity, output location
+
+                     else
+                        Write_Info_Char (Left);
+                        Trunit := Get_Source_Unit (Sloc (Tref));
+
+                        if Trunit /= Curxu then
+                           Write_Info_Nat (Dependency_Num (Trunit));
+                           Write_Info_Char ('|');
+                        end if;
+
+                        Write_Info_Nat
+                          (Int (Get_Logical_Line_Number (Sloc (Tref))));
+
+                        declare
+                           Ent  : Entity_Id := Tref;
+                           Kind : constant Entity_Kind := Ekind (Ent);
+                           Ctyp : Character := Xref_Entity_Letters (Kind);
+
+                        begin
+                           if Ctyp = '+'
+                             and then Present (Full_View (Ent))
+                           then
+                              Ent := Underlying_Type (Ent);
+
+                              if Present (Ent) then
+                                 Ctyp := Xref_Entity_Letters (Ekind (Ent));
+                              end if;
+                           end if;
+
+                           Write_Info_Char (Ctyp);
+                        end;
+
+                        Write_Info_Nat
+                          (Int (Get_Column_Number (Sloc (Tref))));
+
+                        --  If the type comes from an instantiation,
+                        --  add the corresponding info.
+
+                        Output_Instantiation_Refs (Sloc (Tref));
+                        Write_Info_Char (Right);
+                     end if;
+                  end if;
+               end Check_Type_Reference;
+
                -------------------------------
                -- Output_Instantiation_Refs --
                -------------------------------
@@ -1078,6 +1403,35 @@ package body Lib.Xref is
                   return;
                end Output_Instantiation_Refs;
 
+               --------------------------
+               -- Output_Overridden_Op --
+               --------------------------
+
+               procedure Output_Overridden_Op (Old_E : Entity_Id) is
+               begin
+                  if Present (Old_E)
+                    and then Sloc (Old_E) /= Standard_Location
+                  then
+                     declare
+                        Loc      : constant Source_Ptr := Sloc (Old_E);
+                        Par_Unit : constant Unit_Number_Type :=
+                                     Get_Source_Unit (Loc);
+                     begin
+                        Write_Info_Char ('<');
+
+                        if Par_Unit /= Curxu then
+                           Write_Info_Nat (Dependency_Num (Par_Unit));
+                           Write_Info_Char ('|');
+                        end if;
+
+                        Write_Info_Nat (Int (Get_Logical_Line_Number (Loc)));
+                        Write_Info_Char ('p');
+                        Write_Info_Nat (Int (Get_Column_Number (Loc)));
+                        Write_Info_Char ('>');
+                     end;
+                  end if;
+               end Output_Overridden_Op;
+
             --  Start of processing for Output_One_Ref
 
             begin
@@ -1134,42 +1488,67 @@ package body Lib.Xref is
                      if Present (Ent) then
                         Ctyp := Fold_Lower (Xref_Entity_Letters (Ekind (Ent)));
                      end if;
+
+                  elsif Is_Generic_Type (Ent) then
+
+                     --  If the type of the entity is a  generic private type
+                     --  there is no usable full view, so retain the indication
+                     --  that this is an object.
+
+                     Ctyp := '*';
                   end if;
 
                   --  Special handling for access parameter
 
-                  if Ekind (Etype (XE.Ent)) = E_Anonymous_Access_Type
-                    and then Is_Formal (XE.Ent)
-                  then
-                     Ctyp := 'p';
+                  declare
+                     K : constant Entity_Kind := Ekind (Etype (XE.Ent));
 
-                  --  Special handling for Boolean
+                  begin
+                     if (K = E_Anonymous_Access_Type
+                           or else
+                         K = E_Anonymous_Access_Subprogram_Type
+                            or else K =
+                         E_Anonymous_Access_Protected_Subprogram_Type)
+                       and then Is_Formal (XE.Ent)
+                     then
+                        Ctyp := 'p';
 
-                  elsif Ctyp = 'e' and then Is_Boolean_Type (Ent) then
-                     Ctyp := 'b';
-                  end if;
-               end if;
+                        --  Special handling for Boolean
 
-               --  Special handling for abstract types and operations.
+                     elsif Ctyp = 'e' and then Is_Boolean_Type (Ent) then
+                        Ctyp := 'b';
+                     end if;
+                  end;
+               end if;
 
-               if Is_Abstract (XE.Ent) then
+               --  Special handling for abstract types and operations
 
+               if Is_Overloadable (XE.Ent)
+                 and then Is_Abstract_Subprogram (XE.Ent)
+               then
                   if Ctyp = 'U' then
                      Ctyp := 'x';            --  abstract procedure
 
                   elsif Ctyp = 'V' then
                      Ctyp := 'y';            --  abstract function
+                  end if;
+
+               elsif Is_Type (XE.Ent)
+                 and then Is_Abstract_Type (XE.Ent)
+               then
+                  if Is_Interface (XE.Ent) then
+                     Ctyp := 'h';
 
                   elsif Ctyp = 'R' then
                      Ctyp := 'H';            --  abstract type
                   end if;
                end if;
 
-               --  Only output reference if interesting type of entity,
-               --  and suppress self references, except for bodies that
-               --  act as specs. Also suppress definitions of body formals
-               --  (we only treat these as references, and the references
-               --  were separately recorded).
+               --  Only output reference if interesting type of entity, and
+               --  suppress self references, except for bodies that act as
+               --  specs. Also suppress definitions of body formals (we only
+               --  treat these as references, and the references were
+               --  separately recorded).
 
                if Ctyp = ' '
                  or else (XE.Loc = XE.Def
@@ -1354,6 +1733,11 @@ package body Lib.Xref is
                            end if;
                         end loop;
 
+                        --  Write out the identifier by copying the exact
+                        --  source characters used in its declaration. Note
+                        --  that this means wide characters will be in their
+                        --  original encoded form.
+
                         for J in
                           Original_Location (Sloc (XE.Ent)) .. P - 1
                         loop
@@ -1389,7 +1773,25 @@ package body Lib.Xref is
                            Rref := Selector_Name (Rref);
                         end if;
 
-                        if Nkind (Rref) /= N_Identifier then
+                        if Nkind (Rref) = N_Identifier
+                          or else Nkind (Rref) = N_Operator_Symbol
+                        then
+                           null;
+
+                        --  For renamed array components, use the array name
+                        --  for the renamed entity, which reflect the fact that
+                        --  in general the whole array is aliased.
+
+                        elsif Nkind (Rref) = N_Indexed_Component then
+                           if Nkind (Prefix (Rref)) = N_Identifier then
+                              Rref := Prefix (Rref);
+                           elsif Nkind (Prefix (Rref)) = N_Expanded_Name then
+                              Rref := Selector_Name (Prefix (Rref));
+                           else
+                              Rref := Empty;
+                           end if;
+
+                        else
                            Rref := Empty;
                         end if;
                      end if;
@@ -1405,66 +1807,63 @@ package body Lib.Xref is
                           (Int (Get_Column_Number (Sloc (Rref))));
                      end if;
 
-                     --  Indicate that the entity is in the unit
-                     --  of the current xref xection.
+                     --  Indicate that the entity is in the unit of the current
+                     --  xref xection.
 
                      Curru := Curxu;
 
-                     --  See if we have a type reference and if so output
-
-                     Get_Type_Reference (XE.Ent, Tref, Left, Right);
+                     --  Write out information about generic parent, if entity
+                     --  is an instance.
 
-                     if Present (Tref) then
+                     if  Is_Generic_Instance (XE.Ent) then
+                        declare
+                           Gen_Par : constant Entity_Id :=
+                                       Generic_Parent
+                                         (Specification
+                                            (Unit_Declaration_Node (XE.Ent)));
+                           Loc     : constant Source_Ptr := Sloc (Gen_Par);
+                           Gen_U   : constant Unit_Number_Type :=
+                                       Get_Source_Unit (Loc);
 
-                        --  Case of standard entity, output name
-
-                        if Sloc (Tref) = Standard_Location then
-                           Write_Info_Char (Left);
-                           Write_Info_Name (Chars (Tref));
-                           Write_Info_Char (Right);
-
-                        --  Case of source entity, output location
-
-                        else
-                           Write_Info_Char (Left);
-                           Trunit := Get_Source_Unit (Sloc (Tref));
-
-                           if Trunit /= Curxu then
-                              Write_Info_Nat (Dependency_Num (Trunit));
+                        begin
+                           Write_Info_Char ('[');
+                           if Curru /= Gen_U then
+                              Write_Info_Nat (Dependency_Num (Gen_U));
                               Write_Info_Char ('|');
                            end if;
 
                            Write_Info_Nat
-                             (Int (Get_Logical_Line_Number (Sloc (Tref))));
-
-                           declare
-                              Ent  : Entity_Id := Tref;
-                              Kind : constant Entity_Kind := Ekind (Ent);
-                              Ctyp : Character := Xref_Entity_Letters (Kind);
+                             (Int (Get_Logical_Line_Number (Loc)));
+                           Write_Info_Char (']');
+                        end;
+                     end if;
 
-                           begin
-                              if Ctyp = '+'
-                                and then Present (Full_View (Ent))
-                              then
-                                 Ent := Underlying_Type (Ent);
+                     --  See if we have a type reference and if so output
 
-                                 if Present (Ent) then
-                                    Ctyp := Xref_Entity_Letters (Ekind (Ent));
-                                 end if;
-                              end if;
+                     Check_Type_Reference (XE.Ent, False);
 
-                              Write_Info_Char (Ctyp);
-                           end;
+                     if Is_Record_Type (XE.Ent)
+                       and then Present (Abstract_Interfaces (XE.Ent))
+                     then
+                        declare
+                           Elmt : Elmt_Id;
 
-                           Write_Info_Nat
-                             (Int (Get_Column_Number (Sloc (Tref))));
+                        begin
+                           Elmt := First_Elmt (Abstract_Interfaces (XE.Ent));
+                           while Present (Elmt) loop
+                              Check_Type_Reference (Node (Elmt), True);
+                              Next_Elmt (Elmt);
+                           end loop;
+                        end;
+                     end if;
 
-                           --  If the type comes from an instantiation,
-                           --  add the corresponding info.
+                     --  If the entity is an overriding operation, write
+                     --  info on operation that was overridden.
 
-                           Output_Instantiation_Refs (Sloc (Tref));
-                           Write_Info_Char (Right);
-                        end if;
+                     if Is_Subprogram (XE.Ent)
+                       and then Is_Overriding_Operation (XE.Ent)
+                     then
+                        Output_Overridden_Op (Overridden_Operation (XE.Ent));
                      end if;
 
                      --  End of processing for entity output