merge vm_version_x86_{32,64}.hpp
Vladimir Kozlov
Vladimir.Kozlov at Sun.COM
Sun Feb 22 12:21:45 PST 2009
Common vm_version_x86.{cpp,hpp} with #ifdef as we did for assembler_x86.?pp
Vladimir
Christian Thalinger wrote:
> On Fri, 2009-02-20 at 11:35 -0500, Paul Hohensee wrote:
>> Feel free. :) In general, if you're changing things somewhere, feel
>> free to refactor.
>
> Great :-) Should I merge them into a common vm_version_x86.{cpp,hpp}
> plus specific vm_version_x86_{32,64}.{cpp,hpp} files or just a single
> vm_version_x86.{cpp,hpp} with #ifdef's?
>
> There would be very few #ifdef's but I would prefer the former approach.
>
> -- Christian
>
More information about the hotspot-dev
mailing list