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

Sergey Nazarkin snazarkin at azul.com
Thu Nov 3 09:44:59 UTC 2016


Dear Chi Vo,

thank you very much for the finding. I’ll commit the fix ASAP. 

WBR,

Sergey Nazarkin




> On 03 Nov 2016, at 11:24, Chi Vo <thegeek108 at gmail.com> 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



More information about the aarch32-port-dev mailing list