merge vm_version_x86_{32,64}.hpp
Vladimir Kozlov
Vladimir.Kozlov at Sun.COM
Sun Feb 22 15:38:01 PST 2009
Actually we are merging x86 source as much as possible
to avoid duplicated changes for 32- and 64-bits.
Steve Goldman started this work and Tom did additional changes.
So Christian's suggestion is in the line with this work.
It is better to have 2 files with few #ifdef then 4 separate file
which have 80% (or more) of the same duplicated code.
Thanks,
Vladimir
David Holmes - Sun Microsystems wrote:
> Looking at the HS sources the tendency has been to separate files rather
> #ifdefs. I know it's not 100% consistent but I'd rather not help swing
> the pendulum back the other way unless there's a really good reason for
> doing so.
>
> David Holmes
>
> Vladimir Kozlov said the following on 02/23/09 06:21:
>> 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