diff options
author | Zack Rusin <zack@tungstengraphics.com> | 2007-10-26 19:12:02 -0400 |
---|---|---|
committer | Zack Rusin <zack@tungstengraphics.com> | 2007-10-26 19:12:56 -0400 |
commit | 789d248558061fe4d65f664d6770a12b90fa2e34 (patch) | |
tree | 51d17829b47536c8a8a0b0c929e8b28532f29ab3 | |
parent | 78c1f8b2decf168d183c52e7b414adb29dd18988 (diff) | |
download | external_mesa3d-789d248558061fe4d65f664d6770a12b90fa2e34.zip external_mesa3d-789d248558061fe4d65f664d6770a12b90fa2e34.tar.gz external_mesa3d-789d248558061fe4d65f664d6770a12b90fa2e34.tar.bz2 |
Hold a stack of temporaries so that we can redeclare them
for all defined functions. Fixes crashes in function calls.
-rw-r--r-- | src/mesa/pipe/llvm/llvmtgsi.cpp | 4 | ||||
-rw-r--r-- | src/mesa/pipe/llvm/storage.cpp | 21 | ||||
-rw-r--r-- | src/mesa/pipe/llvm/storage.h | 3 |
3 files changed, 27 insertions, 1 deletions
diff --git a/src/mesa/pipe/llvm/llvmtgsi.cpp b/src/mesa/pipe/llvm/llvmtgsi.cpp index cfeb19e..af60232 100644 --- a/src/mesa/pipe/llvm/llvmtgsi.cpp +++ b/src/mesa/pipe/llvm/llvmtgsi.cpp @@ -508,19 +508,21 @@ translate_instruction(llvm::Module *module, case TGSI_OPCODE_BGNSUB: { instr->bgnSub(instno, storage); storage->setCurrentBlock(instr->currentBlock()); + storage->pushTemps(); return; } break; case TGSI_OPCODE_ENDLOOP2: { instr->endLoop(); storage->setCurrentBlock(instr->currentBlock()); - storage->popArguments(); return; } break; case TGSI_OPCODE_ENDSUB: { instr->endSub(); storage->setCurrentBlock(instr->currentBlock()); + storage->popArguments(); + storage->popTemps(); return; } break; diff --git a/src/mesa/pipe/llvm/storage.cpp b/src/mesa/pipe/llvm/storage.cpp index 88ef671..7300cdf 100644 --- a/src/mesa/pipe/llvm/storage.cpp +++ b/src/mesa/pipe/llvm/storage.cpp @@ -369,3 +369,24 @@ void Storage::popArguments() m_CONST = arg.cst; m_argStack.pop(); } + +void Storage::pushTemps() +{ + m_tempStack.push(m_temps); + std::vector<llvm::Value*> oldTemps = m_temps; + m_temps = std::vector<llvm::Value*>(32); + int i = 0; + for (std::vector<llvm::Value*>::iterator itr = oldTemps.begin(); + itr != oldTemps.end(); ++itr) { + if (*itr) { + declareTemp(i); + } + ++i; + } +} + +void Storage::popTemps() +{ + m_temps = m_tempStack.top(); + m_tempStack.pop(); +} diff --git a/src/mesa/pipe/llvm/storage.h b/src/mesa/pipe/llvm/storage.h index b8d6eb0..eeadaa0 100644 --- a/src/mesa/pipe/llvm/storage.h +++ b/src/mesa/pipe/llvm/storage.h @@ -84,6 +84,8 @@ public: void pushArguments(llvm::Value *out, llvm::Value *in, llvm::Value *constPtr); void popArguments(); + void pushTemps(); + void popTemps(); private: llvm::Value *maskWrite(llvm::Value *src, int mask, llvm::Value *templ); @@ -122,6 +124,7 @@ private: llvm::Value *cst; }; std::stack<Args> m_argStack; + std::stack<std::vector<llvm::Value*> > m_tempStack; }; #endif |