8198950: AArch64: org.openjdk.jcstress.tests.varhandles.DekkerTest fails
authoradinn
Mon, 05 Mar 2018 13:18:55 +0000
changeset 49356 5043644f817f
parent 49355 30f39453f0a8
child 49357 aaedb8343784
8198950: AArch64: org.openjdk.jcstress.tests.varhandles.DekkerTest fails Summary: Fix breakage to AArch64 code generation caused by JDK-8181211 Reviewed-by: thartmann, roland, kvn, vlivanov, jrose
src/hotspot/share/opto/library_call.cpp
--- a/src/hotspot/share/opto/library_call.cpp	Tue Mar 06 08:16:45 2018 +0100
+++ b/src/hotspot/share/opto/library_call.cpp	Mon Mar 05 13:18:55 2018 +0000
@@ -2578,7 +2578,8 @@
       // the one that guards them: pin the Load node
       LoadNode::ControlDependency dep = LoadNode::Pinned;
       Node* ctrl = control();
-      if (adr_type->isa_instptr()) {
+      // non volatile loads may be able to float
+      if (!need_mem_bar && adr_type->isa_instptr()) {
         assert(adr_type->meet(TypePtr::NULL_PTR) != adr_type->remove_speculative(), "should be not null");
         intptr_t offset = Type::OffsetBot;
         AddPNode::Ideal_base_and_offset(adr, &_gvn, offset);