Request for reviews (L): 7063628: Use cbcond on T4

Vladimir Kozlov vladimir.kozlov at oracle.com
Wed Jul 13 20:44:37 PDT 2011


http://cr.openjdk.java.net/~kvn/7063628/webrev

Fixed 7063628: Use cbcond on T4

Added new MacroAssembler instructions cmp_and_br() and old branch instructions 
are modified to use cbcond on T4 if distance is small (2K bytes). Most of the 
rest changes are usage of these new branch instructions in Interpreter and C1.

The prototype was done by Tom and I took some of his additional fixes. The 
formssel.cpp change is a bug fix where the MatchNode equality test wasn't 
recursing so it would mistakenly return true for complex matches. Added a fake 
label for branches generated in temp buffer by MachNode::emit_size(), added 
assert into .ad files to check label.

There was problem in is_in_wdisp16_range() (and in initial implementation of 
use_cbc()) which calls target(L) before emitting branch. Non-bound (forward 
branch) labels record current pc() as a branch address and later try to patch 
instruction (which could be 'cmp') at that address.

Removed unused code in check_klass_subtype_fast_path().

Modified vm_version string: has_ prefix is removed and only v8 or v9 will be 
printed as well only niagara_plus or niagara.

Tested on T4 with CTW, nsk, VM and java invoke regression tests.


More information about the hotspot-compiler-dev mailing list