RFR: 8017480: Move copying of jfr files to closed makefile
David Holmes
david.holmes at oracle.com
Tue Jun 25 12:55:47 UTC 2013
Erik,
Thanks for fixing this in this way - much appreciated.
One thing I want to flag. In hotspot we have to add .PHONY entries for
all files that we might -include, otherwise make tries to create them by
executing a bunch of implicit rules, one of which triggers premature
execution of the precompiled headers logic.
Now the main build may not have this problem, but I would think make
wastes a lot of time attempting to build a missing include file when we
don't want it to even attempt that. So the .PHONY should improve
performance.
David
On 24/06/2013 9:57 PM, Erik Joelsson wrote:
> Please review this simple patch, removing copying of jfr files from
> CopyFiles.gmk and adding an optional include of a custom CopyFiles.gmk.
>
> http://cr.openjdk.java.net/~erikj/8017114/webrev.jdk.01/
>
> /Erik
More information about the build-dev
mailing list