Make sure that we setup a barrier before installing the jitted code

Also make sure we remove the setup call frame at the proper moment.
lmars · Oct 5, 2012 · 3fdd8b5 · 3fdd8b5
1 parent 880ecc3
commit 3fdd8b5
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/vm/llvm/state.cpp b/vm/llvm/state.cpp
@@ -328,6 +328,7 @@ namespace rubinius {
 
           jit::RuntimeDataHolder* rd = jit.context().runtime_data_holder();
 
+          atomic::memory_barrier();
           ls_->start_method_update();
 
           if(!req->is_block()) {
@@ -687,12 +688,12 @@ namespace rubinius {
 
       state->set_call_frame(call_frame);
       state->gc_independent(gct);
-      state->set_call_frame(0);
 
       wait_cond.wait(wait_mutex);
 
       wait_mutex.unlock();
       state->gc_dependent();
+      state->set_call_frame(0);
       // if(config().jit_inline_debug) {
         // if(block) {
           // log() << "JIT: compiled block inside: "