RFR(XS): 8185567: fix hsdis cpu to architecture mapping on various Linux platforms
Tim Bell
tim.bell at oracle.com
Thu Aug 3 20:49:27 UTC 2017
> On 8/1/17 5:32 AM, David Buck wrote:
>> Hi!
>>
>> It was suggested that I also loop in build-dev to the review of this
>> proposed change.
The build Makefile change looks good.
Tim
On 08/03/17 13:08, Vladimir Ivanov wrote:> Looks good!
>
> Best regards,
> Vladimir Ivanov
>
>>
>> Cheers,
>> -Buck
>>
>> On 2017/07/31 22:06, David Buck wrote:
>>> Hi!
>>>
>>> Would you please review this small set of very simple build fixes to
>>> hsdis?
>>>
>>> bug report: https://bugs.openjdk.java.net/browse/JDK-8185567
>>>
>>> proposed fix: http://cr.openjdk.java.net/~dbuck/8185567.0/
>>>
>>> With these changes, hsdis should now correctly build out of the box
>>> (no manual editing of files needed) on Linux with any of the
>>> following return values of "uname -m":
>>>
>>> i386
>>> i686
>>> amd64
>>> x86_64
>>> *arm*
>>> aarch64
>>>
>>> I made it a point to *not* break anything that previously worked. So,
>>> for example, aarch64 environments will still compile for aarch64 even
>>> when the build target is "all" and not "all64". While forcing it to
>>> try and build a 32-bit binary would be more consistent with what
>>> happens with "make all" on linux-amd64 or solaris-sparcv9, I decided
>>> to leave things as they are in case any aarch64 users depend on the
>>> current behavior. Similarly, an attempt to run "make all" on
>>> linux-sparcv9 will continue to try and build against whatever the
>>> return value of "uname -m" was. Of course running "make all64" now
>>> does the right thing and targets "sparcv9".
>>>
>>> Cheers,
>>> -Buck
>>>
>>>
More information about the build-dev
mailing list