Re: [PATCH, doc RFA] Remove support for "target m32rsdi" and "target mips/pmon/ddb/rockhopper/lsi"

Maciej W. Rozycki
2016-05-04T01:03:09Z

On Wed, 4 May 2016, Trevor Saunders wrote:

> > > Especially the heuristic
> > > unwinder bits look highly suspicious to me, where the variable is updated
> > > as the analysis proceeds. It could be that these variables can indeed go,
> > > but perhaps something is missing that should be there.
>
> perhaps, my guess is that it was necessary at one time, but isn't
> anymore.

That can be tracked down rather easily now that we have GIT rather than
CVS driving our repository, but I'm more concerned they might be symptoms
of unfinished or buggy code. This is hairy stuff (by definition, actually
-- where there's no DWARF information available we should try PDR records
first, where available, before resorting, in desperation, to heuristics).

Maciej