From 83b24b627d5d699068f32dab00df6b943ec5d591 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Pawe=C5=82=20Bylica?= <pawel.bylica@imapp.pl>
Date: Mon, 27 Oct 2014 16:00:41 +0100
Subject: [PATCH] Moving CALLDATA data from Ext to Runtime [#81470252]

---
 libevmjit/Compiler.cpp |  2 +-
 libevmjit/Ext.cpp      |  4 +---
 libevmjit/Ext.h        |  1 -
 libevmjit/Runtime.cpp  | 10 +++++++++-
 libevmjit/Runtime.h    |  4 +++-
 5 files changed, 14 insertions(+), 7 deletions(-)

diff --git a/libevmjit/Compiler.cpp b/libevmjit/Compiler.cpp
index 7f6aba051..a872a0097 100644
--- a/libevmjit/Compiler.cpp
+++ b/libevmjit/Compiler.cpp
@@ -720,7 +720,7 @@ void Compiler::compileBasicBlock(BasicBlock& basicBlock, bytesConstRef bytecode,
 			auto srcIdx = stack.pop();
 			auto reqBytes = stack.pop();
 
-			auto srcPtr = ext.calldata();
+			auto srcPtr = _runtimeManager.getCallData();
 			auto srcSize = _runtimeManager.get(RuntimeData::CallDataSize);
 
 			memory.copyBytes(srcPtr, srcSize, srcIdx, destMemIdx, reqBytes);
diff --git a/libevmjit/Ext.cpp b/libevmjit/Ext.cpp
index 1adaf741c..b75cd3195 100644
--- a/libevmjit/Ext.cpp
+++ b/libevmjit/Ext.cpp
@@ -52,7 +52,6 @@ Ext::Ext(RuntimeManager& _runtimeManager):
 	m_arg8 = m_builder.CreateAlloca(i256Ty, nullptr, "ext.arg8");
 
 	Type* elements[] = {
-		i8PtrTy, // byte* calldata
 		i8PtrTy, // byte* code
 
 	};
@@ -100,8 +99,7 @@ Value* Ext::getDataElem(unsigned _index, const Twine& _name)
 	return m_builder.CreateLoad(valuePtr);
 }
 
-Value* Ext::calldata() { return getDataElem(0, "calldata"); }
-Value* Ext::code() { return getDataElem(1, "code"); }
+Value* Ext::code() { return getDataElem(0, "code"); }
 
 Value* Ext::calldataload(Value* _index)
 {
diff --git a/libevmjit/Ext.h b/libevmjit/Ext.h
index 62938f3cb..f798ec549 100644
--- a/libevmjit/Ext.h
+++ b/libevmjit/Ext.h
@@ -20,7 +20,6 @@ public:
 	llvm::Value* store(llvm::Value* _index);
 	void setStore(llvm::Value* _index, llvm::Value* _value);
 
-	llvm::Value* calldata();
 	llvm::Value* code();
 
 	llvm::Value* balance(llvm::Value* _address);
diff --git a/libevmjit/Runtime.cpp b/libevmjit/Runtime.cpp
index 21fcfaddb..523d5ea90 100644
--- a/libevmjit/Runtime.cpp
+++ b/libevmjit/Runtime.cpp
@@ -22,7 +22,8 @@ llvm::StructType* RuntimeData::getType()
 	{
 		llvm::Type* elems[] =
 		{
-			llvm::ArrayType::get(Type::i256, _size)
+			llvm::ArrayType::get(Type::i256, _size),
+			Type::BytePtr
 		};
 		type = llvm::StructType::create(elems, "RuntimeData");
 	}
@@ -75,6 +76,7 @@ Runtime::Runtime(u256 _gas, ExtVMFace& _ext):
 	set(RuntimeData::Difficulty, _ext.currentBlock.difficulty);
 	set(RuntimeData::GasLimit, _ext.currentBlock.gasLimit);
 	set(RuntimeData::CodeSize, _ext.code.size());	// TODO: Use constant
+	m_data.callData = _ext.data.data();
 }
 
 Runtime::~Runtime()
@@ -157,6 +159,12 @@ llvm::Value* RuntimeManager::get(Instruction _inst)
 	}
 }
 
+llvm::Value* RuntimeManager::getCallData()
+{
+	auto ptr = getBuilder().CreateStructGEP(getRuntimePtr(), 1, "calldataPtr");
+	return getBuilder().CreateLoad(ptr, "calldata");
+}
+
 llvm::Value* RuntimeManager::getGas()
 {
 	return get(RuntimeData::Gas);
diff --git a/libevmjit/Runtime.h b/libevmjit/Runtime.h
index a4bb89427..d8f1a2624 100644
--- a/libevmjit/Runtime.h
+++ b/libevmjit/Runtime.h
@@ -45,6 +45,7 @@ struct RuntimeData
 	};
 
 	i256 elems[_size];
+	byte const* callData;
 
 	static llvm::StructType* getType();
 };
@@ -89,7 +90,8 @@ public:
 
 	llvm::Value* get(RuntimeData::Index _index);
 	llvm::Value* get(Instruction _inst);
-	llvm::Value* getGas();
+	llvm::Value* getGas();	// TODO: Remove
+	llvm::Value* getCallData();
 	void setGas(llvm::Value* _gas);
 
 private: