[aarch64-port-dev ] RFR: 8146886: aarch64: fails to build following 8136525 and 8139864

Andrew Haley aph at redhat.com
Tue Jan 12 14:07:17 UTC 2016


On 01/12/2016 02:01 PM, Edward Nevill wrote:
> OK to push?

OK with one change:

+  for (int i = 0; i < ((int)JavaThread::stack_shadow_zone_size() / os::vm_page_size()) - 1; i++) {

should be

+  for (int i = 0; i < (int)(JavaThread::stack_shadow_zone_size() / os::vm_page_size()) - 1; i++) {

I'm sure it doesn't really matter when an int is 32 bits, but I can't
see any good reason to do a narrow division here.

Andrew.



More information about the aarch64-port-dev mailing list