Remove the Segment back-pointer from SegInfo. The only place it was

being used was in resolve_redir(), and due to the way resolve_redir()
is called, the involved test was always failing anyway.  So we lose
nothing by removing it except some complexity -- there is no longer a
circularity between Segments and SegInfos.







git-svn-id: svn://svn.valgrind.org/valgrind/trunk@4019

coregrind/m_debuginfo/priv_symtab.h

@@ -171,7 +171,6 @@ extern void ML_(ppCfiSI)   ( CfiSI* );
 struct _SegInfo {
    struct _SegInfo* next;	/* list of SegInfos */
 
-   Segment	*seg;		/* first segment we're mapped out of */
    Int		ref;
 
    /* Description of the mapped segment. */

coregrind/m_debuginfo/symtab.c

@@ -1648,8 +1648,6 @@ SegInfo *VG_(read_seg_symbols) ( Segment *seg )
    si->cfisi_size = si->cfisi_used = 0;
    si->cfisi_minaddr = si->cfisi_maxaddr = 0;
 
-   si->seg = seg;
-
    si->stab_typetab = NULL;
 
    si->plt_start  = si->plt_size  = 0;

coregrind/m_redir.c

@@ -210,11 +210,6 @@ static Bool resolve_redir(CodeRedirect *redir, const SegInfo *si)
    Bool resolved;
 
    vg_assert(si != NULL);
-   vg_assert(si->seg != NULL);
-
-   /* no redirection from Valgrind segments */
-   if (si->seg->flags & SF_VALGRIND)
-      return False;
 
    resolved = from_resolved(redir);
    vg_assert(!resolved);