review (M) for 6879063: SA should use hsdis for disassembly

[Volker Simonis]

And here just another small addition to the README file:

Binutils don't build with Sun Studio 11 because SS11 doesn't
understand the __FUNCTION__ macros which are used there. Either
replace them with  __func__ or use GCC or SS 12 for compilation.

You'd probably better submit this change before I come up with some
more hairsplitting:)


On 9/18/09, Tom Rodriguez <Thomas.Rodriguez at sun.com> wrote:
> That should be http://cr.openjdk.java.net/~never/6879063/
>
>  tom
>
>
>  On Sep 17, 2009, at 4:17 PM, Tom Rodriguez wrote:
>
>
> > http://javaweb.sfbay.sun.com/~never/webrev/6879063/
> >
> > The SA has Java based disassemblers for x86 and sparc but not for
> > amd64.  Instead of porting to amd64 we should switch over to using
> > hsdis for it like the JVM does.  This required a new entry point into
> > hsdis, decode_instructions_virtual, which separates the address of the
> > code being disassembled from the buffer containing the code.  The
> > existing uses of decode_instructions have been updated to use the new
> > interface and the SA Disassembler has Java native methods that call
> > into hsdis and call back up to Java to perform the disassembly.  I
> > also updated the disassembly printing code to more closely match the
> > JVM though we still don't print out a lot things like oops and relocs
> > in the SA.
> >
> > I deleted all the old disassembler logic since it's incompatible with
> > the new disassembly interface.  I also blew away the moribund dbx
> > based SA interface and few other dead files.  In the end I deleted
> > around 22000 lines of source.
> >
> > Tested by dumping full assembly from core files.
> >
>
>