Script Interpreter 64bit portablity: Defer stack variable copies till…

… later.

We need to defer stack variables copies untill after they have been
allocated on the stack.

With this patch, the game is now playable natively on 64bit linux
systems for at least the first level and a half.
But there are some graphical glitches.

neo/game/script/Script_Compiler.cpp

@@ -2217,6 +2217,11 @@ void idCompiler::ParseFunctionDef( idTypeDef *returnType, const char *name ) {
 		}
 	}
 
+	// And now that the stack variables are sorted out, deal with all those copies we have been deferring.
+	for (int i = 0; i < func->deferredCopies.Num(); i++) {		
+		func->deferredCopies[ i ]->value = func->deferredCopies[ i ]->copyOf->value;
+	}
+
 	// check if we should call the super class destructor
 	if ( oldscope->TypeDef()->Inherits( &type_object ) && !idStr::Icmp( name, "destroy" ) ) {
 		idTypeDef *superClass;

neo/game/script/Script_Program.cpp

@@ -1254,9 +1254,10 @@ idVarDef *idProgram::AllocDef( idTypeDef *type, const char *name, idVarDef *scop
 		if ( !strcmp( name, RESULT_STRING ) ) {
 			// <RESULT> vector defs don't need the _x, _y and _z components
 			assert( scope->Type() == ev_function );
-			def->value.stackOffset	= scope->value.functionPtr->locals;
+			def->value.stackOffset	= -1;
 			def->initialized		= idVarDef::stackVariable;
-			scope->value.functionPtr->locals += type->Size();
+			scope->value.functionPtr->stackVars.Append(def);
+
 		} else if ( scope->TypeDef()->Inherits( &type_object ) ) {
 			idTypeDef	newtype( ev_field, NULL, "float field", 0, &type_float );
 			idTypeDef	*type = GetType( newtype, true );
@@ -1289,8 +1290,16 @@ idVarDef *idProgram::AllocDef( idTypeDef *type, const char *name, idVarDef *scop
 			def_z = AllocDef( &type_float, element, scope, constant );
 
 			// point the vector def to the x coordinate
-			def->value			= def_x->value;
 			def->initialized	= def_x->initialized;
+			def->copyOf = def_x;
+
+			if(def->initialized == idVarDef::stackVariable) {
+				// The variable is allocated on the stack, we need to defer the copy till later.
+				scope->value.functionPtr->deferredCopies.Append(def);
+			} else {
+				def->value = def_x->value;
+
+			}
 		}
 	} else if ( scope->TypeDef()->Inherits( &type_object ) ) {
 		//
@@ -1307,6 +1316,7 @@ idVarDef *idProgram::AllocDef( idTypeDef *type, const char *name, idVarDef *scop
 		// them on the stack until the entire function has been parsed
 		def->initialized		= idVarDef::stackVariable;
 		scope->value.functionPtr->stackVars.Append(def);
+		def->value.stackOffset	= -1;
 	} else {
 		//
 		// global variable

neo/game/script/Script_Program.h

@@ -26,6 +26,8 @@ If you have questions concerning this license or the applicable additional terms
 ===========================================================================
 */
 
+#include <utility>
+
 #ifndef __SCRIPT_PROGRAM_H__
 #define __SCRIPT_PROGRAM_H__
 
@@ -70,6 +72,7 @@ class function_t {
 	int					filenum; 			// source file defined in
 	idList<int>			parmSize;
 	idList<idVarDef *>		stackVars;
+	idList<idVarDef *> 		deferredCopies;
 };
 
 typedef union eval_s {
@@ -317,6 +320,7 @@ class idVarDef {
 	varEval_t				value;
 	idVarDef *				scope; 			// function, namespace, or object the var was defined in
 	int						numUsers;		// number of users if this is a constant
+	idVarDef *				copyOf;
 
 	typedef enum {
 		uninitialized, initializedVariable, initializedConstant, stackVariable