Pls review 6813274 (M)
Paul Hohensee
Paul.Hohensee at Sun.COM
Mon Mar 16 05:51:00 PDT 2009
Thanks for the review. I'll remove the profiled targets.
Tom suggests that instead of adding many new targets, I use KERNEL=1
on the command line, as the 64-bit builds use LP64=1. I'd need to keep the
kernel targets though, because the otherwise the jdk build would have to
specify KERNEL=1 like it specifies ARCH_DATA_MODEL=64 now
(it turns ARCH_DATA_MODEL=64 into LP64=1). I'll try that out
and see how it goes.
Paul
John Rose wrote:
> On Mar 13, 2009, at 11:06 AM, Paul Hohensee wrote:
>
>> Webrev and comments at
>>
>> http://cr.openjdk.java.net/~phh/6813274/6813274/webrev.00/
>
> It looks fine.
>
> Now that we are adding more buildtree targets, it is a perfect time to
> get rid of the 'profiled' flavor, since you are changing the set of
> buildtree targets.
>
> I made the 'profiled' target at the Dawn of Java Time, in the old days
> when profiling code on Unix required special compilation options. It
> was lightly used then and is (AFAIK) never used now.
>
> If someone is desperate to make a build with special profiling flags,
> they can set the same environmental variables that profiled.make
> does. It is the same use case as for anybody that wants to add
> compile-time flags to the mix; use EXTRA_CFLAGS, etc. But having
> 'profiled' as a separately defined target is useless noise.
>
> (One annoyance continually reminds me of the existence of 'profiled':
> I keep bumping into it with pathname completion, when I am aiming at
> the product directory.)
>
> -- John
More information about the hotspot-dev
mailing list