graal-jvmci-8: src/share/vm/opto/compile.cpp comparison

comparison src/share/vm/opto/compile.cpp @ 3252:92add02409c9

Merge

author	jmasa
date	Fri, 08 Apr 2011 14:19:50 -0700
parents	7e88bdae86ec e1162778c1c8
children	5d046bf49ce7

comparison

equal deleted inserted replaced

-:4f978fb6c81a
+:92add02409c9
 // Put top into the hash table ASAP.
 initial_gvn()->transform_no_reclaim(top());
 // Set up tf(), start(), and find a CallGenerator.
-CallGenerator* cg;
+CallGenerator* cg = NULL;
 if (is_osr_compilation()) {
 const TypeTuple *domain = StartOSRNode::osr_domain();
 const TypeTuple *range = TypeTuple::make_range(method()->signature());
 init_tf(TypeFunc::make(domain, range));
 StartNode* s = new (this, 2) StartOSRNode(root(), domain);
 // Normal case.
 init_tf(TypeFunc::make(method()));
 StartNode* s = new (this, 2) StartNode(root(), tf()->domain());
 initial_gvn()->set_type_bottom(s);
 init_start(s);
-float past_uses = method()->interpreter_invocation_count();
+if (method()->intrinsic_id() == vmIntrinsics::_Reference_get && UseG1GC) {
-float expected_uses = past_uses;
+// With java.lang.ref.reference.get() we must go through the
-cg = CallGenerator::for_inline(method(), expected_uses);
+// intrinsic when G1 is enabled - even when get() is the root
+// method of the compile - so that, if necessary, the value in
+// the referent field of the reference object gets recorded by
+// the pre-barrier code.
+// Specifically, if G1 is enabled, the value in the referent
+// field is recorded by the G1 SATB pre barrier. This will
+// result in the referent being marked live and the reference
+// object removed from the list of discovered references during
+// reference processing.
+cg = find_intrinsic(method(), false);
+}
+if (cg == NULL) {
+float past_uses = method()->interpreter_invocation_count();
+float expected_uses = past_uses;
+cg = CallGenerator::for_inline(method(), expected_uses);
+}
 }
 if (failing())  return;
 if (cg == NULL) {
 record_method_not_compilable_all_tiers("cannot parse method");
 return;
 static bool oop_offset_is_sane(const TypeInstPtr* tp) {
 ciInstanceKlass *k = tp->klass()->as_instance_klass();
 // Make sure the offset goes inside the instance layout.
 return k->contains_field_offset(tp->offset());
 // Note that OffsetBot and OffsetTop are very negative.
+}
+// Eliminate trivially redundant StoreCMs and accumulate their
+// precedence edges.
+static void eliminate_redundant_card_marks(Node* n) {
+assert(n->Opcode() == Op_StoreCM, "expected StoreCM");
+if (n->in(MemNode::Address)->outcnt() > 1) {
+// There are multiple users of the same address so it might be
+// possible to eliminate some of the StoreCMs
+Node* mem = n->in(MemNode::Memory);
+Node* adr = n->in(MemNode::Address);
+Node* val = n->in(MemNode::ValueIn);
+Node* prev = n;
+bool done = false;
+// Walk the chain of StoreCMs eliminating ones that match.  As
+// long as it's a chain of single users then the optimization is
+// safe.  Eliminating partially redundant StoreCMs would require
+// cloning copies down the other paths.
+while (mem->Opcode() == Op_StoreCM && mem->outcnt() == 1 && !done) {
+if (adr == mem->in(MemNode::Address) &&
+val == mem->in(MemNode::ValueIn)) {
+// redundant StoreCM
+if (mem->req() > MemNode::OopStore) {
+// Hasn't been processed by this code yet.
+n->add_prec(mem->in(MemNode::OopStore));
+} else {
+// Already converted to precedence edge
+for (uint i = mem->req(); i < mem->len(); i++) {
+// Accumulate any precedence edges
+if (mem->in(i) != NULL) {
+n->add_prec(mem->in(i));
+}
+}
+// Everything above this point has been processed.
+done = true;
+}
+// Eliminate the previous StoreCM
+prev->set_req(MemNode::Memory, mem->in(MemNode::Memory));
+assert(mem->outcnt() == 0, "should be dead");
+mem->disconnect_inputs(NULL);
+} else {
+prev = mem;
+}
+mem = prev->in(MemNode::Memory);
+}
+}
 }
 //------------------------------final_graph_reshaping_impl----------------------
 // Implement items 1-5 from final_graph_reshaping below.
 static void final_graph_reshaping_impl( Node *n, Final_Reshape_Counts &frc ) {
 case Op_StoreF:
 case Op_LoadF:
 frc.inc_float_count();
 goto handle_mem;
+case Op_StoreCM:
+{
+// Convert OopStore dependence into precedence edge
+Node* prec = n->in(MemNode::OopStore);
+n->del_req(MemNode::OopStore);
+n->add_prec(prec);
+eliminate_redundant_card_marks(n);
+}
+// fall through
 case Op_StoreB:
 case Op_StoreC:
-case Op_StoreCM:
 case Op_StorePConditional:
 case Op_StoreI:
 case Op_StoreL:
 case Op_StoreIConditional:
 case Op_StoreLConditional:

Mercurial > hg > graal-jvmci-8

comparison src/share/vm/opto/compile.cpp @ 3252:92add02409c9