Latest repo fails to build on OS X

[Greg Lewis]

On Thu, Nov 27, 2008 at 09:01:42PM -0500, Michael Franz wrote:
> I updated from the repo to day to test out all the merged code.  I get the
> following errors when trying to build:
> /Users/mfranz/developer/openjdk-bsd/repos/2008-11-27/hotspot/src/cpu/x86/vm/interp_masm_x86_32.hpp:
> In member function 'void
> InterpreterMacroAssembler::empty_expression_stack()':
> /Users/mfranz/developer/openjdk-bsd/repos/2008-11-27/hotspot/src/cpu/x86/vm/interp_masm_x86_32.hpp:123:
> error: call of overloaded 'movptr(Address, int)' is ambiguous
> /Users/mfranz/developer/openjdk-bsd/repos/2008-11-27/hotspot/src/cpu/x86/vm/assembler_x86.hpp:2010:
> note: candidates are: void MacroAssembler::movptr(Address, intptr_t)
> /Users/mfranz/developer/openjdk-bsd/repos/2008-11-27/hotspot/src/cpu/x86/vm/assembler_x86.hpp:2012:
> note:                 void MacroAssembler::movptr(Address, RegisterImpl*)
> make[6]: *** [incls/_precompiled.incl.gch] Error 1
> 
> I have not investigated yet, were any of the merged code dealing with this?
> 
> I was able to build the November 24 updates.

All we have since then are merges from the main OpenJDK repository.

In reference to your follow ups, I can build the code on both FreeBSD/i386
and FreeBSD/amd64, so that covers both 32 bit and 64 bit platforms.

I also see from your followups that Mac OS X is 32 bit, so this code will
end up being

movl(dst, src);

no matter which variant the compiler picks.  It should pick the intptr_t
version though.  In fact, on FreeBSD/i386 intptr_t is typedef'ed to
int32_t, so these are the same.

I wonder how intptr_t is defined on Mac OS X and what version of gcc is
being used?  You could try changing the cast to (intptr_t) instead of
(int32_t), since it should basically be the same.

-- 
Greg Lewis                          Email   : glewis at eyesbeyond.com
Eyes Beyond                         Web     : http://www.eyesbeyond.com
Information Technology              FreeBSD : glewis at FreeBSD.org