Changeset 436 in dev for trunk/abdev/BasicCompiler64/CParameter.cpp

Timestamp:

Mar 15, 2008, 3:33:36 PM (17 years ago)

Author:

dai_9181

Message:

関数の戻り値の構造体など、一時メモリに保持された構造体のメンバに直接アクセスした場合、その一時メモリの解放が正常に行われないバグを修正（64bit版も修正した）。

File:

• trunk/abdev/BasicCompiler64/CParameter.cpp (modified) (8 diffs)

trunk/abdev/BasicCompiler64/CParameter.cpp

@@ -65 +65 @@
                 int reg = REG_RAX;
                 Type calcType;
-                NumOpe( &reg, Parms[i2], dummyType, calcType );
+                bool isNeedHeapFreeStructure;
+                NumOpe( &reg, Parms[i2], dummyType, calcType, &isNeedHeapFreeStructure );
 
                 if( !calcType.IsStruct() ){
@@ -86 +87 @@
                     i2);
 
-                if( result ){
+                if( result )
+                {
                     useTempParameters[i2] = true;
+                    isNeedFreeStructures[i2] = isNeedHeapFreeStructure;
                     useTempObject = true;
 
@@ -109 +112 @@
                 // 構造体の一時メモリ
 
-                //メモリを解放する
-
-                pobj_sf->pop( REG_RCX );
-
-                //call free
-                extern const UserProc *pSub_free;
-                compiler.codeGenerator.op_call(pSub_free);
-            }
-            else{
+                if( isNeedFreeStructures[i2] )
+                {
+                    //メモリを解放する
+
+                    pobj_sf->pop( REG_RCX );
+
+                    //call free
+                    extern const UserProc *pSub_free;
+                    compiler.codeGenerator.op_call(pSub_free);
+                }
+                else
+                {
+                    pobj_sf->pop();
+                }
+            }
+            else
+            {
                 pobj_sf->pop();
                 pobj_sf->pop();
@@ -148 +159 @@
 
         Type calcType;
-        BOOL bUseHeap;
+        bool isNeedHeapFreeStructure;
         int temp_reg=REG_RAX;
         NumOpe( &temp_reg,
@@ -154 +165 @@
             baseType,
             calcType,
-            &bUseHeap );
+            &isNeedHeapFreeStructure );
 
 
@@ -169 +180 @@
             baseType,
             calcType,
-            &RelativeVar,bUseHeap);
+            &RelativeVar,isNeedHeapFreeStructure);
 
 
@@ -282 +293 @@
             temp_reg=reg;
 
-            BOOL bCalcUseHeap;
             Type calcType;
-            if( !NumOpe( &temp_reg, Parms[i2], dummyType, calcType, &bCalcUseHeap ) ){
+            bool isNeedHeapFreeStructure;
+            if( !NumOpe( &temp_reg, Parms[i2], dummyType, calcType, &isNeedHeapFreeStructure ) ){
                 break;
             }
@@ -294 +305 @@
                     !dummyType.GetClass().IsEqualsOrSubClass( &calcType.GetClass() ) ){
                         //キャスト演算子のオーバーロードに対応する
-                        CallCastOperatorProc( reg, calcType, bCalcUseHeap,dummyType );
+                        CallCastOperatorProc( reg, calcType, isNeedHeapFreeStructure,dummyType );
                 }
             }