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

Timestamp:

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

Author:

dai_9181

Message:

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

File:

• trunk/abdev/BasicCompiler32/CParameter.cpp (modified) (6 diffs)

trunk/abdev/BasicCompiler32/CParameter.cpp

@@ -65 +65 @@
                 //変数ではないとき
                 Type calcType;
-                BOOL bUseHeap;
-                NumOpe( Parms[i2], dummyType, calcType, &bUseHeap );
+                bool isNeedHeapFreeStructure;
+                NumOpe( Parms[i2], dummyType, calcType, &isNeedHeapFreeStructure );
                 //↑ここでスタックに積む
 
@@ -86 +86 @@
                     i2);
 
-                if( result && bUseHeap ){
+                if( result )
+                {
                     useTempParameters[i2] = true;
+                    isNeedFreeStructures[i2] = true;
                     useTempObject = true;
 
@@ -107 +109 @@
     for(int i2=ParmsNum-1;i2>=0;i2--){
         if( useTempParameters[i2] ){
-            if( types[i2].IsStruct() ){
+            if( types[i2].IsStruct() )
+            {
                 // 構造体の一時メモリ
 
-                //メモリを解放する
-
-                //call free
-                extern const UserProc *pSub_free;
-                compiler.codeGenerator.op_call(pSub_free);
-            }
-            else{
-                if( types[i2].Is64() ){
+                if( isNeedFreeStructures[i2] )
+                {
+                    //メモリを解放する
+
+                    //call free
+                    extern const UserProc *pSub_free;
+                    compiler.codeGenerator.op_call(pSub_free);
+                }
+                else
+                {
+                    //pop ... 参照を消す
+                    compiler.codeGenerator.op_add_esp( PTR_SIZE );
+                }
+            }
+            else
+            {
+                if( types[i2].Is64() )
+                {
                     //pop ... 参照を消す
                     //pop ... 上位32ビット
@@ -123 +136 @@
                     compiler.codeGenerator.op_add_esp( PTR_SIZE * 3 );
                 }
-                else{
+                else
+                {
                     //pop ... 参照を消す
                     //pop ... 値を消す
@@ -150 +164 @@
 
     Type calcType;
-    BOOL bUseHeap;
+    bool isNeedHeapFreeStructure;
     NumOpe( expression,
         baseType,
         calcType,
-        &bUseHeap );
+        &isNeedHeapFreeStructure );
 
     // ※スタックにある二つのデータ（コピー先、コピー元）の値を必要とする
-    SetStructVariable( baseType, calcType, bUseHeap );
+    SetStructVariable( baseType, calcType, isNeedHeapFreeStructure );
 }
 
@@ -234 +248 @@
             }
 
-            BOOL bCalcUseHeap;
             Type calcType;
-            if( !NumOpe( Parms[i2], dummyType, calcType, &bCalcUseHeap ) ){
+            bool isNeedHeapFreeStructure;
+            if( !NumOpe( Parms[i2], dummyType, calcType, &isNeedHeapFreeStructure ) )
+            {
                 break;
             }
 
-            if( calcType.IsObject() ){
+            if( calcType.IsObject() )
+            {
                 if( !dummyType.IsObject()
                     ||
                     dummyType.IsObject() &&
-                    !dummyType.GetClass().IsEqualsOrSubClass( &calcType.GetClass() ) ){
+                    !dummyType.GetClass().IsEqualsOrSubClass( &calcType.GetClass() ) )
+                {
                         //キャスト演算子のオーバーロードに対応する
-                        CallCastOperatorProc( calcType, bCalcUseHeap,dummyType );
+                        CallCastOperatorProc( calcType, isNeedHeapFreeStructure,dummyType );
                 }
             }