RFR: JDK-8087195: Support building hotspot with devkits on Macosx

Erik Joelsson erik.joelsson at oracle.com
Thu Jun 11 13:11:06 UTC 2015


Hello again,

I missed cleaning the path correctly when testing this so missed another 
needed change. When calling "lipo", the hotspot makefiles should use the 
variable supplied by configure. Here is an updated patch.

Webrev: http://cr.openjdk.java.net/~erikj/8087195/webrev.hotspot.01/

/Erik

On 2015-06-11 12:53, Erik Joelsson wrote:
> Hello,
>
> Please review this small makefile tweak. When using a devkit to build 
> hotspot on Macosx, the dtrace command gets confused and tries to use 
> the wrong preprocessor. I've fixed this by splitting out the running 
> of the preprocessor to a separate call. I've verified by comparing the 
> generated header files with and without the patch.
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8087195
> Patch inline:
> diff -r 11af3990d56c make/bsd/makefiles/dtrace.make
> --- a/make/bsd/makefiles/dtrace.make
> +++ b/make/bsd/makefiles/dtrace.make
> @@ -263,14 +263,19 @@
>  $(DtraceOutDir):
>      mkdir $(DtraceOutDir)
>
> +# When building using a devkit, dtrace cannot find the correct 
> preprocessor so
> +# we run it explicitly before runing dtrace.
>  $(DtraceOutDir)/hotspot.h: $(DTRACE_COMMON_SRCDIR)/hotspot.d | 
> $(DtraceOutDir)
> -    $(QUIETLY) $(DTRACE_PROG) $(DTRACE_OPTS) -C -I. -h -o $@ -s 
> $(DTRACE_COMMON_SRCDIR)/hotspot.d
> +    $(QUIETLY) $(CC) -E $(DTRACE_OPTS) -I. -x c 
> $(DTRACE_COMMON_SRCDIR)/hotspot.d > $(DtraceOutDir)/hotspot.d
> +    $(QUIETLY) $(DTRACE_PROG) -h -o $@ -s $(DtraceOutDir)/hotspot.d
>
>  $(DtraceOutDir)/hotspot_jni.h: $(DTRACE_COMMON_SRCDIR)/hotspot_jni.d 
> | $(DtraceOutDir)
> -    $(QUIETLY) $(DTRACE_PROG) $(DTRACE_OPTS) -C -I. -h -o $@ -s 
> $(DTRACE_COMMON_SRCDIR)/hotspot_jni.d
> +    $(QUIETLY) $(CC) -E $(DTRACE_OPTS) -I. -x c 
> $(DTRACE_COMMON_SRCDIR)/hotspot_jni.d > $(DtraceOutDir)/hotspot_jni.d
> +    $(QUIETLY) $(DTRACE_PROG) -h -o $@ -s $(DtraceOutDir)/hotspot_jni.d
>
>  $(DtraceOutDir)/hs_private.h: $(DTRACE_COMMON_SRCDIR)/hs_private.d | 
> $(DtraceOutDir)
> -    $(QUIETLY) $(DTRACE_PROG) $(DTRACE_OPTS) -C -I. -h -o $@ -s 
> $(DTRACE_COMMON_SRCDIR)/hs_private.d
> +    $(QUIETLY) $(CC) -E $(DTRACE_OPTS) -I. -x c 
> $(DTRACE_COMMON_SRCDIR)/hs_private.d > $(DtraceOutDir)/hs_private.d
> +    $(QUIETLY) $(DTRACE_PROG) -h -o $@ -s $(DtraceOutDir)/hs_private.d
>
>  dtrace_gen_headers: $(DtraceOutDir)/hotspot.h 
> $(DtraceOutDir)/hotspot_jni.h $(DtraceOutDir)/hs_private.h
>
>
> /Erik




More information about the build-dev mailing list