RFR: 7200065 Cross-compilation changes to support the new-build
david.holmes at oracle.com
Fri Sep 21 18:14:11 PDT 2012
On 22/09/2012 2:50 AM, Kelly O'Hair wrote:
> What bothers me on this adlc.make change is that there is no comment in the Makefile as to why ARCHFLAG is filtered out
> of the options. Why is this necessary?
Fredrik will need to fill in the detail but I believe it is to deal with
32-bit builds on 64-bit platforms. The adlc binary will be run on the
current build-host so need not be compiled for 32-bit. That said I'm not
sure what harm it would do.
> The defs.make file changes look fine.
> On Sep 20, 2012, at 11:40 PM, David Holmes wrote:
>> These are Fredrik's contributions from build-infra being moved to mainline (to join the changes Kelly pushed the other day). There is a slight change to Fredrik's original changeset due to a bug with the old build (PATH_SEP wouldn't get set) - and it's simpler to use ?:= than ifdef.
>> I'm acting as one reviewer but would welcome other eyes.
More information about the hotspot-dev