Skip to content

Commit

Permalink
8246256: GenerateLinkOptData should not mutate the interim or bootstr…
Browse files Browse the repository at this point in the history
…ap JDK

Reviewed-by: erikj, ihse
  • Loading branch information
Fedor Burdun authored and cl4es committed Jun 1, 2020
1 parent ad7dafb commit f3e027c
Showing 1 changed file with 5 additions and 3 deletions.
8 changes: 5 additions & 3 deletions make/GenerateLinkOptData.gmk
Original file line number Diff line number Diff line change
Expand Up @@ -69,10 +69,12 @@ $(CLASSLIST_FILE): $(INTERIM_IMAGE_DIR)/bin/java$(EXE_SUFFIX) $(CLASSLIST_JAR)
-Duser.language=en -Duser.country=US \
-cp $(SUPPORT_OUTPUTDIR)/classlist.jar \
build.tools.classlist.HelloClasslist $(LOG_DEBUG)
$(GREP) -v HelloClasslist [email protected] > $(INTERIM_IMAGE_DIR)/lib/classlist
$(GREP) -v HelloClasslist [email protected] > $@.interim
$(FIXPATH) $(INTERIM_IMAGE_DIR)/bin/java -Xshare:dump \
-XX:[email protected] -XX:[email protected] \
-Xmx128M -Xms128M $(LOG_INFO)
$(FIXPATH) $(INTERIM_IMAGE_DIR)/bin/java -XX:[email protected] \
$(FIXPATH) $(INTERIM_IMAGE_DIR)/bin/java -XX:[email protected] \
-XX:[email protected] -XX:[email protected] \
-Djava.lang.invoke.MethodHandle.TRACE_RESOLVE=true \
-Duser.language=en -Duser.country=US \
--module-path $(SUPPORT_OUTPUTDIR)/classlist.jar \
Expand All @@ -86,7 +88,7 @@ $(CLASSLIST_FILE): $(INTERIM_IMAGE_DIR)/bin/java$(EXE_SUFFIX) $(CLASSLIST_JAR)
$(CAT) $(LINK_OPT_DIR)/stderr $(JLI_TRACE_FILE) ; \
exit $$exitcode \
)
$(GREP) -v HelloClasslist [email protected] > $@
$(GREP) -v HelloClasslist [email protected].2 > $@

# The jli trace is created by the same recipe as classlist. By declaring these
# dependencies, make will correctly rebuild both jli trace and classlist
Expand Down

0 comments on commit f3e027c

Please sign in to comment.