Add comment explaining why scope loads can't move later.

jruby · Oct 28, 2014 · c93c7bb · c93c7bb
1 parent 579b8f2
commit c93c7bb
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/core/src/main/java/org/jruby/ir/IRBuilder.java b/core/src/main/java/org/jruby/ir/IRBuilder.java
@@ -1657,6 +1657,9 @@ private IRMethod defineMethodInner(MethodDefNode defNode, IRMethod method, IRSco
 
         addInstr(method, new ReceiveSelfInstr(method.getSelf()));
 
+
+        // These instructions need to be toward the top of the method because they may both be needed for
+        // processing optional arguments as in def foo(a = Object).
         // Set %current_scope = <current-scope>
         // Set %current_module = isInstanceMethod ? %self.metaclass : %self
         int nearestScopeDepth = parent.getNearestModuleReferencingScopeDepth();