ARM: Rewrite disassembler to use hsdis

[Andrew Hughes]

----- Original Message -----
> On 01/23/2012 02:37 PM, Andrew Hughes wrote:
> > 
> > I'm very wary about introducing Makefile changes into a release
> > branch
> > at such a late stage.  How does these changes affect other
> > architectures?
> 
> Not at all.
> 

These chunks would certainly seem to:

+-CFLAGS		+= -fPIC
++CFLAGS		+= -fPIC -g

+-CPPFLAGS	+= -I$(BINUTILSDIR)/include -I$(BINUTILS)/bfd -I$(TARGET_DIR)/bfd
++CPPFLAGS	+= -I$(BINUTILSDIR)/include -I$(BINUTILSDIR)/bfd -I$(TARGET_DIR)/bfd

+-	if [ ! -f $@ ]; then cd $(TARGET_DIR); make all-opcodes; fi
++	if [ ! -f $@ ]; then cd $(TARGET_DIR); make all-opcodes "CFLAGS=$(CFLAGS)"; fi

As far as I can tell from the hard-to-read patch-within-a-patch, this part isn't within
an ARM #ifdef.

Have you posted this upstream yet?  I'd be interested to hear Kelly's thoughts.

> > Is hsdis part of a regular build?
> 
> No, hsdis is never built in any OpenJDK release.
> 

Ok, so how it is built?  Just via icedtea-debug?

> Andrew.
> 

-- 
Andrew :)

Free Java Software Engineer
Red Hat, Inc. (http://www.redhat.com)

PGP Key: 248BDC07 (https://keys.indymedia.org/)
Fingerprint = EC5A 1F5E C0AD 1D15 8F1F  8F91 3B96 A578 248B DC07