RFR: JDK-8166748 Clean out Windows IA64 support

Bob Vandette bob.vandette at oracle.com
Thu Jun 8 12:27:58 UTC 2017


The register code area support was only added for Windows IA64.

  // Give OS a chance to register generated code area.
  // This is used on Windows 64 bit platforms to register
  // Structured Exception Handlers for our generated code.
  os::register_code_area((char*)low_bound(), (char*)high_bound());

The dependency checker support was for all IA64 platforms.

Bob.

> On Jun 8, 2017, at 8:25 AM, George Triantafillou <george.triantafillou at oracle.com> wrote:
> 
> Hi Bob,
> 
> On 6/8/2017 8:12 AM, Bob Vandette wrote:
>> I don’t see these changes in your webrev but there are a few other changes that
>> were added specifically for IA64.
>> 
>> The os::register_code_area function and dependency checking support was added (depChecker_{arch}.?pp).
>> If it’s your desire to remove all traces of IA64, these should be removed as well.
> This change is targeted to Windows.  I'll file a followup CR to remove all traces of IA64 if there's not one already.
> 
> Thanks for the review.
> 
> -George
>> 
>> Bob.
>> 
>> 
>>> On Jun 7, 2017, at 2:50 PM, George Triantafillou <george.triantafillou at oracle.com> wrote:
>>> 
>>> Please review this fix to clean out Windows IA64 support:
>>> 
>>> JBS: https://bugs.openjdk.java.net/browse/JDK-8166748
>>> open webrev (jdk): http://cr.openjdk.java.net/~gtriantafill/8166748-webrev/jdk/webrev/index.html <http://cr.openjdk.java.net/%7Egtriantafill/8166748-webrev/jdk/webrev/index.html>
>>> open webrev (hotspot): http://cr.openjdk.java.net/~gtriantafill/8166748-webrev/hotspot/webrev/index.html <http://cr.openjdk.java.net/%7Egtriantafill/8166748-webrev/hotspot/webrev/index.html>
>>> 
>>> Please note that this issue is specifically for removing Windows IA64 support.
>>> 
>>> Built and tested on Windows x64.  Thanks.
>>> 
>>> -George
>>> 
> 



More information about the core-libs-dev mailing list