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