hg: aarch32-port/jdk8u/hotspot: 8168849: aarch32: support soft-float platform

[Chi Vo (The Geek)]

Hi Ed, Sergey,

Thank you for your quick response.

For Ed's question, I haven't signed the OCA (Oracle Contributor Agreement).

‎Thanks & Best Regards,
Chi Vo

Sent from my BlackBerry 10 smartphone.
  Original Message  
From: Edward Nevill
Sent: Thursday, November 3, 2016 17:12
To: Chi Vo; aarch32-port-dev at openjdk.java.net; snazarkin at azul.com
Reply To: edward.nevill at gmail.com
Subject: Re: hg: aarch32-port/jdk8u/hotspot: 8168849: aarch32: support soft-float platform

Hi Chi Vo,

Thanks for pointing this out.

Have you, or the company you work for, signed the OCA (Oracle Contributor Agreement)?

All the best,
Ed.

On Thu, 2016-11-03 at 15:24 +0700, Chi Vo wrote:
> Dear Snazarki,
> 
> --- a/src/share/vm/c1/c1_LinearScan.cpp	Tue Oct 04 13:21:19 2016 +0300
> +++
> b/src/share/vm/c1/c1_LinearScan.cpp	Thu Oct 27 18:42:29 2016 +0300
> @@
> -32,6 +32,7 @@ 
> #include "c1/c1_LinearScan.hpp"
> 
> #include "c1/c1_ValueStack.hpp"
> 
> #include "utilities/bitMap.inline.hpp"
> +#include
> "vm_version_aarch32.hpp"
> 
> #ifdef TARGET_ARCH_x86
> 
> # include "vmreg_x86.inline.hpp"
> 
> #endif @@
> -196,10 +197,10 @@
> 
> }
> 
> 
> I think this line +#include "vm_version_aarch32.hpp" must be put in block
> code if *#ifdef TARGET_ARCH_aarch32* like that to avoid compiling error
> when compile for other platform.
> (In my case, its compile error for x86)
> 
> #ifdef TARGET_ARCH_aarch32
> # include "vmreg_aarch32.inline.hpp"
> # include "vm_version_aarch32.hpp"
> #endif
> 
> 
> Best Regards,
> Chi Vo