RFR: 8013393: Merge templateTable_x86 32 and 64 bit files into one file.
John Rose
john.r.rose at oracle.com
Wed Mar 4 02:13:28 UTC 2015
On Mar 3, 2015, at 1:39 PM, Max Ockner <max.ockner at oracle.com> wrote:
>
> I have made all of the changes recommended by Chris and John. I realize there are plenty of further cleanups and optimizations that I could make, but I think this is nearing a good stopping point for now. We can always clean it up more later. What do you think?
>
> new webrev: cr.openjdk.java.net/~coleenp/8013393 <http://cr.openjdk.java.net/~coleenp/8013393>
> tested with nsk.jvmti.testlist
I think the webrev is: cr.openjdk.java.net/~coleenp/8013393.02
It looks good. The balance of shared code, #ifdefs, and LP64 macros is very reasonable now.
The template '_new' uses register allocations from 32-bit on 64-bit,
and so there is a slight chance of regression there.
I didn't notice that of the other templates.
You can count me as a reviewer.
— John
More information about the hotspot-dev
mailing list