RFR(M): 8195776: [x86,sparc] A row of minor fixes and enhancements.

Lindenmaier, Goetz goetz.lindenmaier at sap.com
Fri Jan 19 16:31:22 UTC 2018


Hi,

I would like to contribute a row of smaller fixes to these two platforms.
These are fixes that were done by us inhouse in the past years.

Please review. I please need a sponsor.
http://cr.openjdk.java.net/~goetz/wr18/8195776-x86SmallFixes/webrev.01

Some details:

Move null check info before load LIR_Assembler::emit_alloc_obj. Doesn't make sense after it where it was followed by a non-storage access instruction.

c1 on x86: fix for assertion hit when running with -XX:-ImplicitDiv0Checks
Symptom1: assert(info->_oop_map == oop_map, "same CodeEmitInfo used for multiple LIR instructions");
Symptom2: DivByZeroStub was emitted twice per idiv/irem
Fix1: create the lir_idiv without debug info (CodeEmitInfo*). When running with -XX:-ImplicitDiv0Checks the actual idiv cannot trap or reach a safepoint.
Fix2: don't call add_debug_info_for_div0() in the lir assembler. It adds another DivByZeroStub to the stub list.

compiledIC:
Data / destination can be reloaded if not volatile causing assertion to fire.

stub_Generator:
A small optimization for G1: Only call static_write_ref_array_pre from array copy stubs if G1 marking is active.

Fix datatypes of flags in platform xx_globals_<cpu>.hpp files.

Tests of this change will run tonight on all the platforms we build. (hotspot jtreg, jck and a row of bigger tests).

Best regards,
  Goetz.


More information about the hotspot-compiler-dev mailing list