[llvm-commits] [llvm] r86440 - /llvm/trunk/lib/ExecutionEngine/Interpreter/ExternalFunctions.cpp

[Nick Lewycky]

Author: nicholas
Date: Sat Nov  7 23:45:04 2009
New Revision: 86440

URL: http://llvm.org/viewvc/llvm-project?rev=86440&view=rev
Log:
We don't need to byteswap, the interpreter assumes the program is running
native anyways. This fixes a crash using %d and similar in a scanf statement.

Modified:
    llvm/trunk/lib/ExecutionEngine/Interpreter/ExternalFunctions.cpp

Modified: llvm/trunk/lib/ExecutionEngine/Interpreter/ExternalFunctions.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/ExecutionEngine/Interpreter/ExternalFunctions.cpp?rev=86440&r1=86439&r2=86440&view=diff

==============================================================================
--- llvm/trunk/lib/ExecutionEngine/Interpreter/ExternalFunctions.cpp (original)
+++ llvm/trunk/lib/ExecutionEngine/Interpreter/ExternalFunctions.cpp Sat Nov  7 23:45:04 2009
@@ -468,8 +468,7 @@
           }
           break;
 
-        case 'e': case 'g': case 'E':
-        case 'f':
+        case 'e': case 'g': case 'E': case 'f':
           if (Long || LongLong) {
             Size = 8; Ty = Type::getDoubleTy(C);
           } else {
@@ -508,9 +507,6 @@
   GenericValue GV;
   GV.IntVal = APInt(32, sscanf(Args[0], Args[1], Args[2], Args[3], Args[4],
                         Args[5], Args[6], Args[7], Args[8], Args[9]));
-  ByteswapSCANFResults(FT->getContext(),
-                       Args[1], Args[2], Args[3], Args[4],
-                       Args[5], Args[6], Args[7], Args[8], Args[9], 0);
   return GV;
 }
 
@@ -526,9 +522,6 @@
   GenericValue GV;
   GV.IntVal = APInt(32, scanf( Args[0], Args[1], Args[2], Args[3], Args[4],
                         Args[5], Args[6], Args[7], Args[8], Args[9]));
-  ByteswapSCANFResults(FT->getContext(),
-                       Args[0], Args[1], Args[2], Args[3], Args[4],
-                       Args[5], Args[6], Args[7], Args[8], Args[9]);
   return GV;
 }