OSDN Git Service

Daily bump.
[pf3gnuchains/gcc-fork.git] / gcc / ada / lib-xref.adb
index d87daec..0e8337f 100644 (file)
@@ -6,7 +6,7 @@
 --                                                                          --
 --                                 B o d y                                  --
 --                                                                          --
---          Copyright (C) 1998-2010, Free Software Foundation, Inc.         --
+--          Copyright (C) 1998-2011, 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- --
@@ -27,7 +27,6 @@ with Atree;    use Atree;
 with Csets;    use Csets;
 with Elists;   use Elists;
 with Errout;   use Errout;
-with Lib.Util; use Lib.Util;
 with Nlists;   use Nlists;
 with Opt;      use Opt;
 with Restrict; use Restrict;
@@ -43,9 +42,9 @@ with Snames;   use Snames;
 with Stringt;  use Stringt;
 with Stand;    use Stand;
 with Table;    use Table;
-with Widechar; use Widechar;
 
 with GNAT.Heap_Sort_G;
+with GNAT.HTable;
 
 package body Lib.Xref is
 
@@ -58,16 +57,13 @@ package body Lib.Xref is
 
    subtype Xref_Entry_Number is Int;
 
-   type Xref_Entry is record
+   type Xref_Key is record
+      --  These are the components of Xref_Entry that participate in hash
+      --  lookups.
+
       Ent : Entity_Id;
       --  Entity referenced (E parameter to Generate_Reference)
 
-      Def : Source_Ptr;
-      --  Original source location for entity being referenced. Note that these
-      --  values are used only during the output process, they are not set when
-      --  the entries are originally built. This is because private entities
-      --  can be swapped when the initial call is made.
-
       Loc : Source_Ptr;
       --  Location of reference (Original_Location (Sloc field of N parameter
       --  to Generate_Reference). Set to No_Location for the case of a
@@ -83,6 +79,30 @@ package body Lib.Xref is
       --  Unit number corresponding to Loc. Value is undefined and not
       --  referenced if Loc is set to No_Location.
 
+      --  The following components are only used for Alfa cross-references
+
+      Ref_Scope : Entity_Id;
+      --  Entity of the closest subprogram or package enclosing the reference
+
+      Ent_Scope : Entity_Id;
+      --  Entity of the closest subprogram or package enclosing the definition,
+      --  which should be located in the same file as the definition itself.
+   end record;
+
+   type Xref_Entry is record
+      Key : Xref_Key;
+
+      Ent_Scope_File : Unit_Number_Type;
+      --  File for entity Ent_Scope
+
+      Def : Source_Ptr;
+      --  Original source location for entity being referenced. Note that these
+      --  values are used only during the output process, they are not set when
+      --  the entries are originally built. This is because private entities
+      --  can be swapped when the initial call is made.
+
+      HTable_Next : Xref_Entry_Number;
+      --  For use only by Static_HTable
    end record;
 
    package Xrefs is new Table.Table (
@@ -93,6 +113,50 @@ package body Lib.Xref is
      Table_Increment      => Alloc.Xrefs_Increment,
      Table_Name           => "Xrefs");
 
+   --------------
+   -- Xref_Set --
+   --------------
+
+   --  We keep a set of xref entries, in order to avoid inserting duplicate
+   --  entries into the above Xrefs table. An entry is in Xref_Set if and only
+   --  if it is in Xrefs.
+
+   Num_Buckets : constant := 2**16;
+
+   subtype Header_Num is Integer range 0 .. Num_Buckets - 1;
+   type Null_Type is null record;
+   pragma Unreferenced (Null_Type);
+
+   function Hash (F : Xref_Entry_Number) return Header_Num;
+
+   function Equal (F1, F2 : Xref_Entry_Number) return Boolean;
+
+   procedure HT_Set_Next (E : Xref_Entry_Number; Next : Xref_Entry_Number);
+
+   function  HT_Next (E : Xref_Entry_Number) return Xref_Entry_Number;
+
+   function Get_Key (E : Xref_Entry_Number) return Xref_Entry_Number;
+
+   pragma Inline (Hash, Equal, HT_Set_Next, HT_Next, Get_Key);
+
+   package Xref_Set is new GNAT.HTable.Static_HTable (
+     Header_Num,
+     Element    => Xref_Entry,
+     Elmt_Ptr   => Xref_Entry_Number,
+     Null_Ptr   => 0,
+     Set_Next   => HT_Set_Next,
+     Next       => HT_Next,
+     Key        => Xref_Entry_Number,
+     Get_Key    => Get_Key,
+     Hash       => Hash,
+     Equal      => Equal);
+
+   ----------------------
+   -- Alfa Information --
+   ----------------------
+
+   package body Alfa is separate;
+
    ------------------------
    --  Local Subprograms --
    ------------------------
@@ -103,14 +167,54 @@ package body Lib.Xref is
    --  cross-reference information rather than at the freeze point of the type
    --  in order to handle late bodies that are primitive operations.
 
+   function Lt (T1, T2 : Xref_Entry) return Boolean;
+   --  Order cross-references
+
+   procedure Add_Entry (Key : Xref_Key; Ent_Scope_File : Unit_Number_Type);
+   --  Add an entry to the tables of Xref_Entries, avoiding duplicates
+
+   ---------------
+   -- Add_Entry --
+   ---------------
+
+   procedure Add_Entry (Key : Xref_Key; Ent_Scope_File : Unit_Number_Type) is
+   begin
+      Xrefs.Increment_Last; -- tentative
+      Xrefs.Table (Xrefs.Last).Key := Key;
+
+      --  Set the entry in Xref_Set, and if newly set, keep the above
+      --  tentative increment.
+
+      if Xref_Set.Set_If_Not_Present (Xrefs.Last) then
+         Xrefs.Table (Xrefs.Last).Ent_Scope_File := Ent_Scope_File;
+         --  Leave Def and HTable_Next uninitialized
+
+         Set_Has_Xref_Entry (Key.Ent);
+
+      --  It was already in Xref_Set, so throw away the tentatively-added
+      --  entry
+
+      else
+         Xrefs.Decrement_Last;
+      end if;
+   end Add_Entry;
+
+   -----------
+   -- Equal --
+   -----------
+
+   function Equal (F1, F2 : Xref_Entry_Number) return Boolean is
+      Result : constant Boolean :=
+                 Xrefs.Table (F1).Key = Xrefs.Table (F2).Key;
+   begin
+      return Result;
+   end Equal;
+
    -------------------------
    -- Generate_Definition --
    -------------------------
 
    procedure Generate_Definition (E : Entity_Id) is
-      Loc  : Source_Ptr;
-      Indx : Nat;
-
    begin
       pragma Assert (Nkind (E) in N_Entity);
 
@@ -141,17 +245,15 @@ package body Lib.Xref is
          and then In_Extended_Main_Source_Unit (E)
          and then not Is_Internal_Name (Chars (E))
       then
-         Xrefs.Increment_Last;
-         Indx := Xrefs.Last;
-         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);
+         Add_Entry
+           ((Ent => E,
+             Loc => No_Location,
+             Typ => ' ',
+             Eun => Get_Source_Unit (Original_Location (Sloc (E))),
+             Lun => No_Unit,
+             Ref_Scope => Empty,
+             Ent_Scope => Empty),
+            Ent_Scope_File => No_Unit);
 
          if In_Inlined_Body then
             Set_Referenced (E);
@@ -234,21 +336,14 @@ package body Lib.Xref is
          return;
       end if;
 
-      --  Ada 2005 (AI-345): For synchronized types generate reference
-      --  to the wrapper that allow us to dispatch calls through their
-      --  implemented abstract interface types.
+      --  Ada 2005 (AI-345): For synchronized types generate reference to the
+      --  wrapper that allow us to dispatch calls through their implemented
+      --  abstract interface types.
 
-      --  The check for Present here is to protect against previously
-      --  reported critical errors.
+      --  The check for Present here is to protect against previously reported
+      --  critical errors.
 
-      if Is_Concurrent_Type (Base_T)
-        and then Present (Corresponding_Record_Type (Base_T))
-      then
-         Prim_List := Primitive_Operations
-                       (Corresponding_Record_Type (Base_T));
-      else
-         Prim_List := Primitive_Operations (Base_T);
-      end if;
+      Prim_List := Primitive_Operations (Base_T);
 
       if No (Prim_List) then
          return;
@@ -278,11 +373,16 @@ package body Lib.Xref is
       Set_Ref : Boolean   := True;
       Force   : Boolean   := False)
    is
-      Indx : Nat;
-      Nod  : Node_Id;
-      Ref  : Source_Ptr;
-      Def  : Source_Ptr;
-      Ent  : Entity_Id;
+      Nod : Node_Id;
+      Ref : Source_Ptr;
+      Def : Source_Ptr;
+      Ent : Entity_Id;
+
+      Actual_Typ : Character := Typ;
+
+      Ref_Scope      : Entity_Id;
+      Ent_Scope      : Entity_Id;
+      Ent_Scope_File : Unit_Number_Type;
 
       Call   : Node_Id;
       Formal : Entity_Id;
@@ -291,6 +391,10 @@ package body Lib.Xref is
       Kind : Entity_Kind;
       --  If Formal is non-Empty, then its Ekind, otherwise E_Void
 
+      function Get_Through_Renamings (E : Entity_Id) return Entity_Id;
+      --  Get the enclosing entity through renamings, which may come from
+      --  source or from the translation of generic instantiations.
+
       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:
@@ -312,6 +416,22 @@ package body Lib.Xref is
       --  exceptions where we do not want to set this flag, see body for
       --  details of these exceptional cases.
 
+      ---------------------------
+      -- Get_Through_Renamings --
+      ---------------------------
+
+      function Get_Through_Renamings (E : Entity_Id) return Entity_Id is
+         Result : Entity_Id := E;
+      begin
+         while Present (Result)
+           and then Is_Object (Result)
+           and then Present (Renamed_Object (Result))
+         loop
+            Result := Get_Enclosing_Object (Renamed_Object (Result));
+         end loop;
+         return Result;
+      end Get_Through_Renamings;
+
       ---------------
       -- Is_On_LHS --
       ---------------
@@ -412,6 +532,16 @@ package body Lib.Xref is
                      return False;
                   end if;
                end if;
+
+            --  A reference to a formal in a named parameter association does
+            --  not make the formal referenced. Formals that are unused in the
+            --  subprogram body are properly flagged as such, even if calls
+            --  elsewhere use named notation.
+
+            elsif Nkind (P) = N_Parameter_Association
+              and then N = Selector_Name (P)
+            then
+               return False;
             end if;
          end if;
 
@@ -447,14 +577,16 @@ package body Lib.Xref is
       --  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).
+      --  In any case we only generate warnings if we are in the extended main
+      --  source unit, and the entity itself is not in the extended main source
+      --  unit, 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)
+        and then In_Extended_Main_Source_Unit (N)
       then
          Check_Restriction (No_Obsolescent_Features, N);
 
@@ -470,7 +602,7 @@ package body Lib.Xref is
         and then Is_Ada_2005_Only (E)
         and then Ada_Version < Ada_2005
         and then Warn_On_Ada_2005_Compatibility
-        and then (Typ = 'm' or else Typ = 'r')
+        and then (Typ = 'm' or else Typ = 'r' or else Typ = 's')
       then
          Error_Msg_NE ("& is only defined in Ada 2005?", N, E);
       end if;
@@ -502,6 +634,7 @@ package body Lib.Xref is
 
       if not In_Extended_Main_Source_Unit (N) then
          if Typ = 'e'
+           or else Typ = 'I'
            or else Typ = 'p'
            or else Typ = 'i'
            or else Typ = 'k'
@@ -840,31 +973,71 @@ package body Lib.Xref is
             return;
          end if;
 
+         --  In Alfa mode, consider the underlying entity renamed instead of
+         --  the renaming, which is needed to compute a valid set of effects
+         --  (reads, writes) for the enclosing subprogram.
+
+         if Alfa_Mode then
+            Ent := Get_Through_Renamings (Ent);
+
+            --  If no enclosing object, then it could be a reference to any
+            --  location not tracked individually, like heap-allocated data.
+            --  Conservatively approximate this possibility by generating a
+            --  dereference, and return.
+
+            if No (Ent) then
+               if Actual_Typ = 'w' then
+                  Alfa.Generate_Dereference (Nod, 'r');
+                  Alfa.Generate_Dereference (Nod, 'w');
+               else
+                  Alfa.Generate_Dereference (Nod, 'r');
+               end if;
+
+               return;
+            end if;
+         end if;
+
          --  Record reference to entity
 
          Ref := Original_Location (Sloc (Nod));
          Def := Original_Location (Sloc (Ent));
 
-         Xrefs.Increment_Last;
-         Indx := Xrefs.Last;
+         if Actual_Typ = 'p'
+           and then Is_Subprogram (N)
+           and then Present (Overridden_Operation (N))
+         then
+            Actual_Typ := 'P';
+         end if;
 
-         Xrefs.Table (Indx).Loc := Ref;
+         if Alfa_Mode then
+            Ref_Scope := Alfa.Enclosing_Subprogram_Or_Package (N);
+            Ent_Scope := Alfa.Enclosing_Subprogram_Or_Package (Ent);
 
-         --  Overriding operations are marked with 'P'
+            --  Since we are reaching through renamings in Alfa mode, we may
+            --  end up with standard constants. Ignore those.
 
-         if Typ = 'p'
-           and then Is_Subprogram (N)
-           and then Is_Overriding_Operation (N)
-         then
-            Xrefs.Table (Indx).Typ := 'P';
+            if Sloc (Ent_Scope) <= Standard_Location
+              or else Def <= Standard_Location
+            then
+               return;
+            end if;
+
+            Ent_Scope_File := Get_Source_Unit (Ent_Scope);
          else
-            Xrefs.Table (Indx).Typ := Typ;
+            Ref_Scope := Empty;
+            Ent_Scope := Empty;
+            Ent_Scope_File := No_Unit;
          end if;
 
-         Xrefs.Table (Indx).Eun := Get_Source_Unit (Def);
-         Xrefs.Table (Indx).Lun := Get_Source_Unit (Ref);
-         Xrefs.Table (Indx).Ent := Ent;
-         Set_Has_Xref_Entry (Ent);
+         Add_Entry
+           ((Ent => Ent,
+             Loc => Ref,
+             Typ => Actual_Typ,
+             Eun => Get_Source_Unit (Def),
+             Lun => Get_Source_Unit (Ref),
+             Ref_Scope => Ref_Scope,
+             Ent_Scope => Ent_Scope),
+            Ent_Scope_File => Ent_Scope_File);
       end if;
    end Generate_Reference;
 
@@ -929,6 +1102,58 @@ package body Lib.Xref is
       end loop;
    end Generate_Reference_To_Generic_Formals;
 
+   -------------
+   -- Get_Key --
+   -------------
+
+   function Get_Key (E : Xref_Entry_Number) return Xref_Entry_Number is
+   begin
+      return E;
+   end Get_Key;
+
+   ----------
+   -- Hash --
+   ----------
+
+   function Hash (F : Xref_Entry_Number) return Header_Num is
+      --  It is unlikely to have two references to the same entity at the same
+      --  source location, so the hash function depends only on the Ent and Loc
+      --  fields.
+
+      XE : Xref_Entry renames Xrefs.Table (F);
+      type M is mod 2**32;
+
+      H : constant M := M (XE.Key.Ent) + 2 ** 7 * M (abs XE.Key.Loc);
+      --  It would be more natural to write:
+      --
+      --    H : constant M := M'Mod (XE.Key.Ent) + 2**7 * M'Mod (XE.Key.Loc);
+      --
+      --  But we can't use M'Mod, because it prevents bootstrapping with older
+      --  compilers. Loc can be negative, so we do "abs" before converting.
+      --  One day this can be cleaned up ???
+
+   begin
+      return Header_Num (H mod Num_Buckets);
+   end Hash;
+
+   -----------------
+   -- HT_Set_Next --
+   -----------------
+
+   procedure HT_Set_Next (E : Xref_Entry_Number; Next : Xref_Entry_Number) is
+   begin
+      Xrefs.Table (E).HTable_Next := Next;
+   end HT_Set_Next;
+
+   -------------
+   -- HT_Next --
+   -------------
+
+   function HT_Next (E : Xref_Entry_Number) return Xref_Entry_Number is
+   begin
+      return Xrefs.Table (E).HTable_Next;
+   end HT_Next;
+
    ----------------
    -- Initialize --
    ----------------
@@ -938,6 +1163,52 @@ package body Lib.Xref is
       Xrefs.Init;
    end Initialize;
 
+   --------
+   -- Lt --
+   --------
+
+   function Lt (T1, T2 : Xref_Entry) return Boolean is
+   begin
+      --  First test: if entity is in different unit, sort by unit
+
+      if T1.Key.Eun /= T2.Key.Eun then
+         return Dependency_Num (T1.Key.Eun) < Dependency_Num (T2.Key.Eun);
+
+      --  Second test: within same unit, sort by entity Sloc
+
+      elsif T1.Def /= T2.Def then
+         return T1.Def < T2.Def;
+
+      --  Third test: sort definitions ahead of references
+
+      elsif T1.Key.Loc = No_Location then
+         return True;
+
+      elsif T2.Key.Loc = No_Location then
+         return False;
+
+      --  Fourth test: for same entity, sort by reference location unit
+
+      elsif T1.Key.Lun /= T2.Key.Lun then
+         return Dependency_Num (T1.Key.Lun) < Dependency_Num (T2.Key.Lun);
+
+      --  Fifth test: order of location within referencing unit
+
+      elsif T1.Key.Loc /= T2.Key.Loc then
+         return T1.Key.Loc < T2.Key.Loc;
+
+      --  Finally, for two locations at the same address, we prefer
+      --  the one that does NOT have the type 'r' so that a modification
+      --  or extension takes preference, when there are more than one
+      --  reference at the same location. As a result, in the case of
+      --  entities that are in-out actuals, the read reference follows
+      --  the modify reference.
+
+      else
+         return T2.Key.Typ = 'r';
+      end if;
+   end Lt;
+
    -----------------------
    -- Output_References --
    -----------------------
@@ -957,7 +1228,7 @@ package body Lib.Xref is
 
       procedure Output_Import_Export_Info (Ent : Entity_Id);
       --  Output language and external name information for an interfaced
-      --  entity, using the format <language, external_name>,
+      --  entity, using the format <language, external_name>.
 
       ------------------------
       -- Get_Type_Reference --
@@ -1163,23 +1434,19 @@ package body Lib.Xref is
    --  Start of processing for Output_References
 
    begin
-      if not Opt.Xref_Active then
-         return;
-      end if;
-
-      --  First we add references to the primitive operations of tagged
-      --  types declared in the main unit.
+      --  First we add references to the primitive operations of tagged types
+      --  declared in the main unit.
 
       Handle_Prim_Ops : declare
          Ent  : Entity_Id;
 
       begin
          for J in 1 .. Xrefs.Last loop
-            Ent := Xrefs.Table (J).Ent;
+            Ent := Xrefs.Table (J).Key.Ent;
 
             if Is_Type (Ent)
               and then Is_Tagged_Type (Ent)
-              and then Ent = Base_Type (Ent)
+              and then Is_Base_Type (Ent)
               and then In_Extended_Main_Source_Unit (Ent)
             then
                Generate_Prim_Op_References (Ent);
@@ -1213,9 +1480,7 @@ package body Lib.Xref is
       Handle_Orphan_Type_References : declare
          J    : Nat;
          Tref : Entity_Id;
-         Indx : Nat;
          Ent  : Entity_Id;
-         Loc  : Source_Ptr;
 
          L, R : Character;
          pragma Warnings (Off, L);
@@ -1232,18 +1497,20 @@ package body Lib.Xref is
 
          procedure New_Entry (E : Entity_Id) is
          begin
-            if Present (E)
-              and then not Has_Xref_Entry (E)
+            pragma Assert (Present (E));
+
+            if not Has_Xref_Entry (Implementation_Base_Type (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);
+               Add_Entry
+                 ((Ent => E,
+                   Loc => No_Location,
+                   Typ => Character'First,
+                   Eun => Get_Source_Unit (Original_Location (Sloc (E))),
+                   Lun => No_Unit,
+                   Ref_Scope => Empty,
+                   Ent_Scope => Empty),
+                  Ent_Scope_File => No_Unit);
             end if;
          end New_Entry;
 
@@ -1256,7 +1523,7 @@ package body Lib.Xref is
 
          J := 1;
          while J <= Xrefs.Last loop
-            Ent := Xrefs.Table (J).Ent;
+            Ent := Xrefs.Table (J).Key.Ent;
             Get_Type_Reference (Ent, Tref, L, R);
 
             if Present (Tref)
@@ -1288,7 +1555,7 @@ package body Lib.Xref is
             if Is_Type (Ent)
               and then Is_Tagged_Type (Ent)
               and then Is_Derived_Type (Ent)
-              and then Ent = Base_Type (Ent)
+              and then Is_Base_Type (Ent)
               and then In_Extended_Main_Source_Unit (Ent)
             then
                declare
@@ -1323,15 +1590,15 @@ package body Lib.Xref is
                      Prim := Parent_Op (Node (Op));
 
                      if Present (Prim) then
-                        Xrefs.Increment_Last;
-                        Indx := Xrefs.Last;
-                        Loc  := Original_Location (Sloc (Prim));
-                        Xrefs.Table (Indx).Ent := Prim;
-                        Xrefs.Table (Indx).Loc := No_Location;
-                        Xrefs.Table (Indx).Eun :=
-                          Get_Source_Unit (Sloc (Prim));
-                        Xrefs.Table (Indx).Lun := No_Unit;
-                        Set_Has_Xref_Entry (Prim);
+                        Add_Entry
+                          ((Ent => Prim,
+                            Loc => No_Location,
+                            Typ => Character'First,
+                            Eun => Get_Source_Unit (Sloc (Prim)),
+                            Lun => No_Unit,
+                            Ref_Scope => Empty,
+                            Ent_Scope => Empty),
+                           Ent_Scope_File => No_Unit);
                      end if;
 
                      Next_Elmt (Op);
@@ -1348,9 +1615,8 @@ package body Lib.Xref is
 
       Output_Refs : declare
 
-         Nrefs : Nat := Xrefs.Last;
-         --  Number of references in table. This value may get reset (reduced)
-         --  when we eliminate duplicate reference entries.
+         Nrefs : constant Nat := Xrefs.Last;
+         --  Number of references in table
 
          Rnums : array (0 .. Nrefs) of Nat;
          --  This array contains numbers of references in the Xrefs table.
@@ -1365,9 +1631,6 @@ package body Lib.Xref is
          Curru : Unit_Number_Type;
          --  Current reference unit for one entity
 
-         Cursrc : Source_Buffer_Ptr;
-         --  Current xref unit source text
-
          Curent : Entity_Id;
          --  Current entity
 
@@ -1384,6 +1647,9 @@ package body Lib.Xref is
          Ctyp : Character;
          --  Entity type character
 
+         Prevt : Character;
+         --  reference kind of previous reference
+
          Tref : Entity_Id;
          --  Type reference
 
@@ -1413,42 +1679,7 @@ package body Lib.Xref is
             T2 : Xref_Entry renames Xrefs.Table (Rnums (Nat (Op2)));
 
          begin
-            --  First test: if entity is in different unit, sort by unit
-
-            if T1.Eun /= T2.Eun then
-               return Dependency_Num (T1.Eun) < Dependency_Num (T2.Eun);
-
-            --  Second test: within same unit, sort by entity Sloc
-
-            elsif T1.Def /= T2.Def then
-               return T1.Def < T2.Def;
-
-            --  Third test: sort definitions ahead of references
-
-            elsif T1.Loc = No_Location then
-               return True;
-
-            elsif T2.Loc = No_Location then
-               return False;
-
-            --  Fourth test: for same entity, sort by reference location unit
-
-            elsif T1.Lun /= T2.Lun then
-               return Dependency_Num (T1.Lun) < Dependency_Num (T2.Lun);
-
-            --  Fifth test: order of location within referencing unit
-
-            elsif T1.Loc /= T2.Loc then
-               return T1.Loc < T2.Loc;
-
-            --  Finally, for two locations at the same address, we prefer
-            --  the one that does NOT have the type 'r' so that a modification
-            --  or extension takes preference, when there are more than one
-            --  reference at the same location.
-
-            else
-               return T2.Typ = 'r';
-            end if;
+            return Lt (T1, T2);
          end Lt;
 
          ----------
@@ -1488,37 +1719,13 @@ package body Lib.Xref is
          for J in 1 .. Nrefs loop
             Rnums (J) := J;
             Xrefs.Table (J).Def :=
-              Original_Location (Sloc (Xrefs.Table (J).Ent));
+              Original_Location (Sloc (Xrefs.Table (J).Key.Ent));
          end loop;
 
          --  Sort the references
 
          Sorting.Sort (Integer (Nrefs));
 
-         --  Eliminate duplicate entries
-
-         declare
-            NR : constant Nat := Nrefs;
-
-         begin
-            --  We need this test for NR because if we force ALI file
-            --  generation in case of errors detected, it may be the case
-            --  that Nrefs is 0, so we should not reset it here
-
-            if NR >= 2 then
-               Nrefs := 1;
-
-               for J in 2 .. NR loop
-                  if Xrefs.Table (Rnums (J)) /=
-                     Xrefs.Table (Rnums (Nrefs))
-                  then
-                     Nrefs := Nrefs + 1;
-                     Rnums (Nrefs) := Rnums (J);
-                  end if;
-               end loop;
-            end if;
-         end;
-
          --  Initialize loop through references
 
          Curxu  := No_Unit;
@@ -1526,25 +1733,17 @@ package body Lib.Xref is
          Curdef := No_Location;
          Curru  := No_Unit;
          Crloc  := No_Location;
+         Prevt  := 'm';
 
          --  Loop to output references
 
          for Refno in 1 .. Nrefs loop
             Output_One_Ref : declare
-               P2  : Source_Ptr;
                Ent : Entity_Id;
 
-               WC  : Char_Code;
-               Err : Boolean;
-               pragma Warnings (Off, WC);
-               pragma Warnings (Off, Err);
-
                XE : Xref_Entry renames Xrefs.Table (Rnums (Refno));
                --  The current entry to be accessed
 
-               P : Source_Ptr;
-               --  Used to index into source buffer to get entity name
-
                Left  : Character;
                Right : Character;
                --  Used for {} or <> or () for type reference
@@ -1713,7 +1912,9 @@ package body Lib.Xref is
 
                      Op := Ultimate_Alias (Old_E);
 
-                  --  Normal case of no alias present
+                  --  Normal case of no alias present. We omit generated
+                  --  primitives like tagged equality, that have no source
+                  --  representation.
 
                   else
                      Op := Old_E;
@@ -1721,6 +1922,7 @@ package body Lib.Xref is
 
                   if Present (Op)
                     and then Sloc (Op) /= Standard_Location
+                    and then Comes_From_Source (Op)
                   then
                      declare
                         Loc      : constant Source_Ptr := Sloc (Op);
@@ -1746,7 +1948,7 @@ package body Lib.Xref is
             --  Start of processing for Output_One_Ref
 
             begin
-               Ent := XE.Ent;
+               Ent := XE.Key.Ent;
                Ctyp := Xref_Entity_Letters (Ekind (Ent));
 
                --  Skip reference if it is the only reference to an entity,
@@ -1755,12 +1957,12 @@ package body Lib.Xref is
                --  consisting only of packages with END lines, where no
                --  entity from the package is actually referenced.
 
-               if XE.Typ = 'e'
+               if XE.Key.Typ = 'e'
                  and then Ent /= Curent
-                 and then (Refno = Nrefs or else
-                             Ent /= Xrefs.Table (Rnums (Refno + 1)).Ent)
-                 and then
-                   not In_Extended_Main_Source_Unit (Ent)
+                 and then (Refno = Nrefs
+                            or else
+                              Ent /= Xrefs.Table (Rnums (Refno + 1)).Key.Ent)
+                 and then not In_Extended_Main_Source_Unit (Ent)
                then
                   goto Continue;
                end if;
@@ -1768,7 +1970,7 @@ package body Lib.Xref is
                --  For private type, get full view type
 
                if Ctyp = '+'
-                 and then Present (Full_View (XE.Ent))
+                 and then Present (Full_View (XE.Key.Ent))
                then
                   Ent := Underlying_Type (Ent);
 
@@ -1786,15 +1988,15 @@ package body Lib.Xref is
                --  For variable reference, get corresponding type
 
                if Ctyp = '*' then
-                  Ent := Etype (XE.Ent);
+                  Ent := Etype (XE.Key.Ent);
                   Ctyp := Fold_Lower (Xref_Entity_Letters (Ekind (Ent)));
 
                   --  If variable is private type, get full view type
 
                   if Ctyp = '+'
-                    and then Present (Full_View (Etype (XE.Ent)))
+                    and then Present (Full_View (Etype (XE.Key.Ent)))
                   then
-                     Ent := Underlying_Type (Etype (XE.Ent));
+                     Ent := Underlying_Type (Etype (XE.Key.Ent));
 
                      if Present (Ent) then
                         Ctyp := Fold_Lower (Xref_Entity_Letters (Ekind (Ent)));
@@ -1812,13 +2014,13 @@ package body Lib.Xref is
                   --  Special handling for access parameters and objects of
                   --  an anonymous access type.
 
-                  if Ekind_In (Etype (XE.Ent),
+                  if Ekind_In (Etype (XE.Key.Ent),
                                E_Anonymous_Access_Type,
                                E_Anonymous_Access_Subprogram_Type,
                                E_Anonymous_Access_Protected_Subprogram_Type)
                   then
-                     if Is_Formal (XE.Ent)
-                       or else Ekind_In (XE.Ent, E_Variable, E_Constant)
+                     if Is_Formal (XE.Key.Ent)
+                       or else Ekind_In (XE.Key.Ent, E_Variable, E_Constant)
                      then
                         Ctyp := 'p';
                      end if;
@@ -1832,8 +2034,8 @@ package body Lib.Xref is
 
                --  Special handling for abstract types and operations
 
-               if Is_Overloadable (XE.Ent)
-                 and then Is_Abstract_Subprogram (XE.Ent)
+               if Is_Overloadable (XE.Key.Ent)
+                 and then Is_Abstract_Subprogram (XE.Key.Ent)
                then
                   if Ctyp = 'U' then
                      Ctyp := 'x';            --  Abstract procedure
@@ -1842,10 +2044,10 @@ package body Lib.Xref is
                      Ctyp := 'y';            --  Abstract function
                   end if;
 
-               elsif Is_Type (XE.Ent)
-                 and then Is_Abstract_Type (XE.Ent)
+               elsif Is_Type (XE.Key.Ent)
+                 and then Is_Abstract_Type (XE.Key.Ent)
                then
-                  if Is_Interface (XE.Ent) then
+                  if Is_Interface (XE.Key.Ent) then
                      Ctyp := 'h';
 
                   elsif Ctyp = 'R' then
@@ -1853,38 +2055,49 @@ package body Lib.Xref is
                   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
+               --  Only output reference if interesting type of entity
+
+               if Ctyp = ' '
+
+               --  Suppress references to object definitions, used for local
+               --  references.
+
+                 or else XE.Key.Typ = 'D'
+                 or else XE.Key.Typ = 'I'
+
+               --  Suppress self references, except for bodies that act as
+               --  specs.
+
+                 or else (XE.Key.Loc = XE.Def
+                           and then
+                             (XE.Key.Typ /= 'b'
+                               or else not Is_Subprogram (XE.Key.Ent)))
+
+               --  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
-                            and then
-                              (XE.Typ /= 'b'
-                                or else not Is_Subprogram (XE.Ent)))
-                 or else (Is_Formal (XE.Ent)
-                            and then Present (Spec_Entity (XE.Ent)))
+                 or else (Is_Formal (XE.Key.Ent)
+                           and then Present (Spec_Entity (XE.Key.Ent)))
                then
                   null;
 
                else
                   --  Start new Xref section if new xref unit
 
-                  if XE.Eun /= Curxu then
+                  if XE.Key.Eun /= Curxu then
                      if Write_Info_Col > 1 then
                         Write_Info_EOL;
                      end if;
 
-                     Curxu := XE.Eun;
-                     Cursrc := Source_Text (Source_Index (Curxu));
+                     Curxu := XE.Key.Eun;
 
                      Write_Info_Initiate ('X');
                      Write_Info_Char (' ');
-                     Write_Info_Nat (Dependency_Num (XE.Eun));
+                     Write_Info_Nat (Dependency_Num (XE.Key.Eun));
                      Write_Info_Char (' ');
-                     Write_Info_Name (Reference_Name (Source_Index (XE.Eun)));
+                     Write_Info_Name
+                       (Reference_Name (Source_Index (XE.Key.Eun)));
                   end if;
 
                   --  Start new Entity line if new entity. Note that we
@@ -1895,14 +2108,14 @@ package body Lib.Xref is
 
                   if No (Curent)
                     or else
-                      (XE.Ent /= Curent
+                      (XE.Key.Ent /= Curent
                          and then
-                           (Name_Change (XE.Ent) or else XE.Def /= Curdef))
+                           (Name_Change (XE.Key.Ent) or else XE.Def /= Curdef))
                   then
-                     Curent := XE.Ent;
+                     Curent := XE.Key.Ent;
                      Curdef := XE.Def;
 
-                     Get_Unqualified_Name_String (Chars (XE.Ent));
+                     Get_Unqualified_Name_String (Chars (XE.Key.Ent));
                      Curlen := Name_Len;
                      Curnam (1 .. Curlen) := Name_Buffer (1 .. Curlen);
 
@@ -2012,70 +2225,33 @@ package body Lib.Xref is
                      --  Output entity name. We use the occurrence from the
                      --  actual source program at the definition point.
 
-                     P := Original_Location (Sloc (XE.Ent));
-
-                     --  Entity is character literal
-
-                     if Cursrc (P) = ''' then
-                        Write_Info_Char (Cursrc (P));
-                        Write_Info_Char (Cursrc (P + 1));
-                        Write_Info_Char (Cursrc (P + 2));
-
-                     --  Entity is operator symbol
-
-                     elsif Cursrc (P) = '"' or else Cursrc (P) = '%' then
-                        Write_Info_Char (Cursrc (P));
-
-                        P2 := P;
-                        loop
-                           P2 := P2 + 1;
-                           Write_Info_Char (Cursrc (P2));
-                           exit when Cursrc (P2) = Cursrc (P);
-                        end loop;
-
-                     --  Entity is identifier
-
-                     else
-                        loop
-                           if Is_Start_Of_Wide_Char (Cursrc, P) then
-                              Scan_Wide (Cursrc, P, WC, Err);
-                           elsif not Identifier_Char (Cursrc (P)) then
-                              exit;
-                           else
-                              P := P + 1;
-                           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
-                           Write_Info_Char (Cursrc (J));
+                     declare
+                        Ent_Name : constant String :=
+                                     Exact_Source_Name (Sloc (XE.Key.Ent));
+                     begin
+                        for C in Ent_Name'Range loop
+                           Write_Info_Char (Ent_Name (C));
                         end loop;
-                     end if;
+                     end;
 
                      --  See if we have a renaming reference
 
-                     if Is_Object (XE.Ent)
-                       and then Present (Renamed_Object (XE.Ent))
+                     if Is_Object (XE.Key.Ent)
+                       and then Present (Renamed_Object (XE.Key.Ent))
                      then
-                        Rref := Renamed_Object (XE.Ent);
+                        Rref := Renamed_Object (XE.Key.Ent);
 
-                     elsif Is_Overloadable (XE.Ent)
-                       and then Nkind (Parent (Declaration_Node (XE.Ent))) =
-                                            N_Subprogram_Renaming_Declaration
+                     elsif Is_Overloadable (XE.Key.Ent)
+                       and then Nkind (Parent (Declaration_Node (XE.Key.Ent)))
+                                           = N_Subprogram_Renaming_Declaration
                      then
-                        Rref := Name (Parent (Declaration_Node (XE.Ent)));
+                        Rref := Name (Parent (Declaration_Node (XE.Key.Ent)));
 
-                     elsif Ekind (XE.Ent) = E_Package
-                       and then Nkind (Declaration_Node (XE.Ent)) =
+                     elsif Ekind (XE.Key.Ent) = E_Package
+                       and then Nkind (Declaration_Node (XE.Key.Ent)) =
                                          N_Package_Renaming_Declaration
                      then
-                        Rref := Name (Declaration_Node (XE.Ent));
+                        Rref := Name (Declaration_Node (XE.Key.Ent));
 
                      else
                         Rref := Empty;
@@ -2128,12 +2304,13 @@ package body Lib.Xref is
                      --  Write out information about generic parent, if entity
                      --  is an instance.
 
-                     if  Is_Generic_Instance (XE.Ent) then
+                     if  Is_Generic_Instance (XE.Key.Ent) then
                         declare
                            Gen_Par : constant Entity_Id :=
                                        Generic_Parent
                                          (Specification
-                                            (Unit_Declaration_Node (XE.Ent)));
+                                            (Unit_Declaration_Node
+                                               (XE.Key.Ent)));
                            Loc     : constant Source_Ptr := Sloc (Gen_Par);
                            Gen_U   : constant Unit_Number_Type :=
                                        Get_Source_Unit (Loc);
@@ -2154,15 +2331,16 @@ package body Lib.Xref is
 
                      --  See if we have a type reference and if so output
 
-                     Check_Type_Reference (XE.Ent, False);
+                     Check_Type_Reference (XE.Key.Ent, False);
 
                      --  Additional information for types with progenitors
 
-                     if Is_Record_Type (XE.Ent)
-                       and then Present (Interfaces (XE.Ent))
+                     if Is_Record_Type (XE.Key.Ent)
+                       and then Present (Interfaces (XE.Key.Ent))
                      then
                         declare
-                           Elmt : Elmt_Id := First_Elmt (Interfaces (XE.Ent));
+                           Elmt : Elmt_Id :=
+                                    First_Elmt (Interfaces (XE.Key.Ent));
                         begin
                            while Present (Elmt) loop
                               Check_Type_Reference (Node (Elmt), True);
@@ -2170,14 +2348,14 @@ package body Lib.Xref is
                            end loop;
                         end;
 
-                     --  For array types, list index types as well.
-                     --  (This is not C, indices have distinct types).
+                     --  For array types, list index types as well. (This is
+                     --  not C, indexes have distinct types).
 
-                     elsif Is_Array_Type (XE.Ent) then
+                     elsif Is_Array_Type (XE.Key.Ent) then
                         declare
                            Indx : Node_Id;
                         begin
-                           Indx := First_Index (XE.Ent);
+                           Indx := First_Index (XE.Key.Ent);
                            while Present (Indx) loop
                               Check_Type_Reference
                                 (First_Subtype (Etype (Indx)), True);
@@ -2189,10 +2367,11 @@ package body Lib.Xref is
                      --  If the entity is an overriding operation, write info
                      --  on operation that was overridden.
 
-                     if Is_Subprogram (XE.Ent)
-                       and then Is_Overriding_Operation (XE.Ent)
+                     if Is_Subprogram (XE.Key.Ent)
+                       and then Present (Overridden_Operation (XE.Key.Ent))
                      then
-                        Output_Overridden_Op (Overridden_Operation (XE.Ent));
+                        Output_Overridden_Op
+                          (Overridden_Operation (XE.Key.Ent));
                      end if;
 
                      --  End of processing for entity output
@@ -2200,12 +2379,17 @@ package body Lib.Xref is
                      Crloc := No_Location;
                   end if;
 
-                  --  Output the reference
+                  --  Output the reference if it is not as the same location
+                  --  as the previous one, or it is a read-reference that
+                  --  indicates that the entity is an in-out actual in a call.
 
-                  if XE.Loc /= No_Location
-                     and then XE.Loc /= Crloc
+                  if XE.Key.Loc /= No_Location
+                    and then
+                      (XE.Key.Loc /= Crloc
+                        or else (Prevt = 'm' and then  XE.Key.Typ = 'r'))
                   then
-                     Crloc := XE.Loc;
+                     Crloc := XE.Key.Loc;
+                     Prevt := XE.Key.Typ;
 
                      --  Start continuation if line full, else blank
 
@@ -2218,25 +2402,26 @@ package body Lib.Xref is
 
                      --  Output file number if changed
 
-                     if XE.Lun /= Curru then
-                        Curru := XE.Lun;
+                     if XE.Key.Lun /= Curru then
+                        Curru := XE.Key.Lun;
                         Write_Info_Nat (Dependency_Num (Curru));
                         Write_Info_Char ('|');
                      end if;
 
-                     Write_Info_Nat  (Int (Get_Logical_Line_Number (XE.Loc)));
-                     Write_Info_Char (XE.Typ);
+                     Write_Info_Nat
+                       (Int (Get_Logical_Line_Number (XE.Key.Loc)));
+                     Write_Info_Char (XE.Key.Typ);
 
-                     if Is_Overloadable (XE.Ent)
-                       and then Is_Imported (XE.Ent)
-                       and then XE.Typ = 'b'
+                     if Is_Overloadable (XE.Key.Ent)
+                       and then Is_Imported (XE.Key.Ent)
+                       and then XE.Key.Typ = 'b'
                      then
-                        Output_Import_Export_Info (XE.Ent);
+                        Output_Import_Export_Info (XE.Key.Ent);
                      end if;
 
-                     Write_Info_Nat (Int (Get_Column_Number (XE.Loc)));
+                     Write_Info_Nat (Int (Get_Column_Number (XE.Key.Loc)));
 
-                     Output_Instantiation_Refs (Sloc (XE.Ent));
+                     Output_Instantiation_Refs (Sloc (XE.Key.Ent));
                   end if;
                end if;
             end Output_One_Ref;
@@ -2249,4 +2434,9 @@ package body Lib.Xref is
       end Output_Refs;
    end Output_References;
 
+begin
+   --  Reset is necessary because Elmt_Ptr does not default to Null_Ptr,
+   --  because it's not an access type.
+
+   Xref_Set.Reset;
 end Lib.Xref;