RFR(XS): 8185567: fix hsdis cpu to architecture mapping on various Linux platforms

Vladimir Ivanov vladimir.x.ivanov at oracle.com
Thu Aug 3 20:08:49 UTC 2017


Looks good!

Best regards,
Vladimir Ivanov

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.
> 
> 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