Re: Zero fails to build on SPARC again, similar to JDK-8186578

2018-04-10 Thread John Paul Adrian Glaubitz
On 04/10/2018 08:04 PM, Erik Joelsson wrote: "JDK-8201236 Straighten out dtrace build logic" Aye, changeset is coming up ;). Indeed, this fixes it! Thanks so much, I was already about to give up ;). We should have been explicit with that parameter in the first place, then Magnus would not

Re: Zero fails to build on SPARC again, similar to JDK-8186578

2018-04-10 Thread Erik Joelsson
On 2018-04-10 10:50, John Paul Adrian Glaubitz wrote: Hi Erik! On 04/10/2018 06:54 PM, Erik Joelsson wrote: I've found the problem. In JvmFeatures.gmk we have: ifeq ($(call check-jvm-feature, zero), true)    JVM_CFLAGS_FEATURES += -DZERO -DCC_INTERP -DZERO_LIBARCH='"$(OPENJDK_TARGET_CPU_LEG

Re: Zero fails to build on SPARC again, similar to JDK-8186578

2018-04-10 Thread John Paul Adrian Glaubitz
Hi Erik! On 04/10/2018 06:54 PM, Erik Joelsson wrote: I've found the problem. In JvmFeatures.gmk we have: ifeq ($(call check-jvm-feature, zero), true)   JVM_CFLAGS_FEATURES += -DZERO -DCC_INTERP -DZERO_LIBARCH='"$(OPENJDK_TARGET_CPU_LEGACY_LIB)"' $(LIBFFI_CFLAGS)   JVM_LIBS_FEATURES += $(LIB

Re: Zero fails to build on SPARC again, similar to JDK-8186578

2018-04-10 Thread Erik Joelsson
I've found the problem. In JvmFeatures.gmk we have: ifeq ($(call check-jvm-feature, zero), true)   JVM_CFLAGS_FEATURES += -DZERO -DCC_INTERP -DZERO_LIBARCH='"$(OPENJDK_TARGET_CPU_LEGACY_LIB)"' $(LIBFFI_CFLAGS)   JVM_LIBS_FEATURES += $(LIBFFI_LIBS)   ifeq ($(OPENJDK_TARGET_CPU), sparcv9)     BUI

Re: Zero fails to build on SPARC again, similar to JDK-8186578

2018-04-10 Thread John Paul Adrian Glaubitz
On 04/10/2018 01:37 PM, John Paul Adrian Glaubitz wrote: @buildd-dev: I need to build memset_with_concurrent_readers_sparc.cpp for Zero on SPARC as the Zero build now bails out with linker errors: Add the source file in question to EXTRA_FILES: glaubitz@deb4g:/srv/glaubitz/hs$ hg diff diff -r

Re: Zero fails to build on SPARC again, similar to JDK-8186578

2018-04-10 Thread John Paul Adrian Glaubitz
On 04/10/2018 01:34 PM, David Holmes wrote: Adding in build-dev. I think the build guys need to help you figure this out. Good idea. @buildd-dev: I need to build memset_with_concurrent_readers_sparc.cpp for Zero on SPARC as the Zero build now bails out with linker errors: === Output from fai

Re: Zero fails to build on SPARC again, similar to JDK-8186578

2018-04-10 Thread David Holmes
Adding in build-dev. I think the build guys need to help you figure this out. Cheers, David On 10/04/2018 9:21 PM, John Paul Adrian Glaubitz wrote: On 04/10/2018 01:00 PM, John Paul Adrian Glaubitz wrote: Trying to add the necessary source there now. Hmm, I tried various ways of adding it n