On 14/09/2018 11:22 AM, Mikael Vidstedt wrote:
On Sep 13, 2018, at 6:17 PM, David Holmes <david.holmes@oracle.com> wrote:
Looks good - though the proof is in the building of course.
The debug-only use of notDouble in src/hotspot/cpu/x86/templateTable_x86.cpp seems somewhat odd.
Would you prefer to see it declared next to the other labels, but guarded with ASSERT?
No what you have is fine. I'm just curious about why the double case is only examined in a non-product build ?? David
Cheers, Mikael
Thanks, David
On 14/09/2018 9:03 AM, Mikael Vidstedt wrote:
Please review this change which removes a bunch of unused Label variables. Would appreciate some help from aarch64/ppc/s390x folks to verify it! bug: https://bugs.openjdk.java.net/browse/JDK-8210676 webrev: http://cr.openjdk.java.net/~mikael/webrevs/8210676/webrev.03/open/webrev/ * Background (from bug) [~dholmes] noticed during the code review of JDK-8210381 that the "Label Egress" variable in macroAssembler_sparc.cpp was unused. It and other unused labels like it should be removed. * About the change I have *not* tried to find and remove *all* unused Label variables, because that turns out to be much harder than it might seem. I may or may not follow up on this work to remove additional unused Label variables later, but before that I’m investigating removal of other unused variables in general. Meanwhile I like to think that this is a reasonable cleanup anyway. * Testing tier1 build&test passes. Cheers, Mikael