[macruby-changes] [921] MacRuby/branches/experimental/roxor.cpp

source_changes at macosforge.org source_changes at macosforge.org
Sat Mar 14 03:22:23 PDT 2009


Revision: 921
          http://trac.macosforge.org/projects/ruby/changeset/921
Author:   vincent.isambart at gmail.com
Date:     2009-03-14 03:22:19 -0700 (Sat, 14 Mar 2009)
Log Message:
-----------
rolling back the previous change
it was fixing a problem but did no seem to work in some cases
needs more investigation

Modified Paths:
--------------
    MacRuby/branches/experimental/roxor.cpp

Modified: MacRuby/branches/experimental/roxor.cpp
===================================================================
--- MacRuby/branches/experimental/roxor.cpp	2009-03-14 10:12:46 UTC (rev 920)
+++ MacRuby/branches/experimental/roxor.cpp	2009-03-14 10:22:19 UTC (rev 921)
@@ -253,7 +253,6 @@
 	Constant *nilVal;
 	Constant *trueVal;
 	Constant *falseVal;
-	Constant *undefVal;
 	Constant *splatArgFollowsVal;
 	const Type *RubyObjTy; 
 	const Type *RubyObjPtrTy; 
@@ -572,7 +571,6 @@
     nilVal = ConstantInt::get(RubyObjTy, Qnil);
     trueVal = ConstantInt::get(RubyObjTy, Qtrue);
     falseVal = ConstantInt::get(RubyObjTy, Qfalse);
-    undefVal = ConstantInt::get(RubyObjTy, Qundef);
     splatArgFollowsVal = ConstantInt::get(RubyObjTy, SPLAT_ARG_FOLLOWS);
     PtrTy = PointerType::getUnqual(Type::Int8Ty);
 
@@ -735,7 +733,7 @@
     do {
 	assert(node->nd_value != NULL);
 
-	Value *isNilInst = new ICmpInst(ICmpInst::ICMP_EQ, iter, undefVal, "", bb);
+	Value *isNilInst = new ICmpInst(ICmpInst::ICMP_EQ, iter, nilVal, "", bb);
 
 	Function *f = bb->getParent();
 	BasicBlock *arg_nil = BasicBlock::Create("arg_nil", f);
@@ -4150,7 +4148,7 @@
 		new_argv[i] = argv[i];
 	    }
 	    else {
-		new_argv[i] = Qundef;
+		new_argv[i] = Qnil;
 	    }
 	}
 	return __rb_vm_rcall(self, node, pimp, -arity, -arity, new_argv);
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.macosforge.org/pipermail/macruby-changes/attachments/20090314/52ac543f/attachment-0001.html>


More information about the macruby-changes mailing list