Changeset 435 in dev for trunk/abdev/BasicCompiler_Common

Timestamp:

Mar 15, 2008, 1:20:13 PM (17 years ago)

Author:

dai_9181

Message:

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

Location:

trunk/abdev/BasicCompiler_Common

Files:

• Compile.cpp (modified) (1 diff)
• include/CodeGenerator.h (modified) (3 diffs)
• src/CommonCodeGenerator.cpp (modified) (1 diff)

trunk/abdev/BasicCompiler_Common/Compile.cpp

@@ -457 +457 @@
         case COM_LET:
             OpcodeCalc(Command+2);
+
             break;
         default:
             OpcodeOthers(Command);
+
+            // コード生成過程で発生した構造体の一時メモリを破棄する
+            compiler.codeGenerator.op_FreeTempStructure();
+
             break;
     }

trunk/abdev/BasicCompiler_Common/include/CodeGenerator.h

@@ -198 +198 @@
     std::vector<long> continueCodePositions;
 
+    // コンパイル中のステップにおいて、構造体の一時オブジェクトの解放が必要かどうか
+    bool isNeedFreeTempStructureInCurrentStep;
+
 public:
 
@@ -215 +218 @@
     CodeGenerator()
         : pNativeCode( 0 )
+        , isNeedFreeTempStructureInCurrentStep( false )
     {
     }
@@ -294 +298 @@
     void op_jmp_exitsub();
     void op_jmp_goto_schedule( const std::string &name, int lineNum, int sourceCodePos );
+    void op_AddNeedFreeTempStructure( int reg );
+    void op_FreeTempStructure();
 
 

trunk/abdev/BasicCompiler_Common/src/CommonCodeGenerator.cpp

@@ -304 +304 @@
     pNativeCode->Put( (long)0 );
 }
+
+void CodeGenerator::op_AddNeedFreeTempStructure( int reg )
+{
+#ifdef _AMD64_
+    //////////////////////////////////////////////////////
+    /////    レジスタ資源のバックアップ
+    {   BACKUP_REGISTER_RESOURCE
+    //////////////////////////////////////////////////////
+
+    //mov rcx,reg
+    compiler.codeGenerator.op_mov_RR( REG_RCX, reg );
+
+    //call _System_AddNeedFreeTempStructure
+    extern const UserProc *pSub_System_AddNeedFreeTempStructure;
+    compiler.codeGenerator.op_call( pSub_System_AddNeedFreeTempStructure );
+
+    /////////////////////////////////////////////
+    //////   レジスタ資源を復元
+        RESTORE_REGISTER_RESOURCE
+    }////////////////////////////////////////////
+#else
+
+    //push useReg（引き続き利用するため、退避しておく）
+    compiler.codeGenerator.op_push( reg );
+
+    //push useReg
+    compiler.codeGenerator.op_push( reg );
+
+    //call _System_AddNeedFreeTempStructure
+    extern const UserProc *pSub_System_AddNeedFreeTempStructure;
+    compiler.codeGenerator.op_call( pSub_System_AddNeedFreeTempStructure );
+
+    //pop useReg（復元する）
+    compiler.codeGenerator.op_pop( reg );
+#endif
+
+    isNeedFreeTempStructureInCurrentStep = true;
+}
+void CodeGenerator::op_FreeTempStructure()
+{
+    if( !isNeedFreeTempStructureInCurrentStep )
+    {
+        // 解放の必要はない
+        return;
+    }
+
+    // call _System_FreeTempStructure
+    extern const UserProc *pSub_System_FreeTempStructure;
+    op_call( pSub_System_FreeTempStructure );
+
+    isNeedFreeTempStructureInCurrentStep = false;
+}