# HG changeset patch
# User Christian Kamm <kamm incasoftware de>
# Date 1225358197 -3600
# Node ID 1ae94fb1dbbd393d05a57d6a0db79c95211a5889
# Parent  ecb429ee0648c429458657ae548f3d3f98535074
Fix accidental double-inreg caused by shared TupleType. Enabled inreg by default.

diff -r ecb429ee0648 -r 1ae94fb1dbbd dmd/mtype.c
--- a/dmd/mtype.c	Thu Oct 30 10:15:42 2008 +0100
+++ b/dmd/mtype.c	Thu Oct 30 10:16:37 2008 +0100
@@ -3454,7 +3454,7 @@
 	    }
 	}
 	if (t->ty == Ttuple)
-	    *pt = t;
+	    *pt = t->syntaxCopy();
 	else
 	    *pt = t->merge();
     }
diff -r ecb429ee0648 -r 1ae94fb1dbbd gen/functions.cpp
--- a/gen/functions.cpp	Thu Oct 30 10:15:42 2008 +0100
+++ b/gen/functions.cpp	Thu Oct 30 10:16:37 2008 +0100
@@ -203,16 +203,18 @@
             // pass first param in EAX if it fits, is not floating point and is not a 3 byte struct.
             // FIXME: struct are not passed in EAX yet
 
+            int n_inreg = f->reverseParams ? n - 1 : 0;
+            Argument* arg = Argument::getNth(f->parameters, n_inreg);
+
             // if there is a implicit context parameter, pass it in EAX
             if (usesthis || usesnest)
             {
                 f->thisAttrs |= llvm::Attribute::InReg;
+                assert((!arg || (arg->llvmAttrs & llvm::Attribute::InReg) == 0) && "can't have two inreg args!");
             }
             // otherwise check the first formal parameter
             else
             {
-                int inreg = f->reverseParams ? n - 1 : 0;
-                Argument* arg = Argument::getNth(f->parameters, inreg);
                 Type* t = arg->type->toBasetype();
 
                 // 32bit ints, pointers, classes, static arrays and AAs
@@ -223,6 +225,7 @@
                     (t->size() <= PTRSIZE))
                 {
                     arg->llvmAttrs |= llvm::Attribute::InReg;
+                    assert((f->thisAttrs & llvm::Attribute::InReg) == 0 && "can't have two inreg args!");
                 }
             }
         }
diff -r ecb429ee0648 -r 1ae94fb1dbbd premake.lua
--- a/premake.lua	Thu Oct 30 10:15:42 2008 +0100
+++ b/premake.lua	Thu Oct 30 10:16:37 2008 +0100
@@ -39,7 +39,7 @@
 
 -- x86 ABI support
 X86_REVERSE_PARAMS = 1
-X86_PASS_IN_EAX = 0 --disabled for now
+X86_PASS_IN_EAX = 1
 
 -- D version - don't change these !!!
 DMDV1 = "1"