[llvm-commits] CVS: llvm/lib/ExecutionEngine/Interpreter/Interpreter.cpp

Reid Spencer reid at x10sys.com
Mon Mar 5 19:06:14 PST 2007



Changes in directory llvm/lib/ExecutionEngine/Interpreter:

Interpreter.cpp updated: 1.38 -> 1.39
---
Log message:

Remove the insufficient code in Interpreter::create that computed the
Target DataLayout incorrectly. For now, we'll trust that the module has
got the correct DataLayout. In the future, this needs to be changed to
tell the TargetData to be "current host".


---
Diffs of the changes:  (+0 -12)

 Interpreter.cpp |   12 ------------
 1 files changed, 12 deletions(-)


Index: llvm/lib/ExecutionEngine/Interpreter/Interpreter.cpp
diff -u llvm/lib/ExecutionEngine/Interpreter/Interpreter.cpp:1.38 llvm/lib/ExecutionEngine/Interpreter/Interpreter.cpp:1.39
--- llvm/lib/ExecutionEngine/Interpreter/Interpreter.cpp:1.38	Sat Mar  3 12:29:16 2007
+++ llvm/lib/ExecutionEngine/Interpreter/Interpreter.cpp	Mon Mar  5 21:05:57 2007
@@ -46,18 +46,6 @@
   // when the module is deleted via the ExistingModuleProvide via EE.
   delete MP;
   
-  // FIXME: This should probably compute the entire data layout
-  std::string DataLayout;
-  int Test = 0;
-  *(char*)&Test = 1;    // Return true if the host is little endian
-  bool isLittleEndian = (Test == 1);
-  DataLayout.append(isLittleEndian ? "e" : "E");
-
-  bool Ptr64 = sizeof(void*) == 8;
-  DataLayout.append(Ptr64 ? "-p:64:64" : "-p:32:32");
-	
-  M->setDataLayout(DataLayout);
-
   return new Interpreter(M);
 }
 






More information about the llvm-commits mailing list