be more careful about sext/zext of i32 to i64, the input element may be modified, so copy it

1 files changed, 2 insertions, 2 deletions

diff --git a/lib/Transforms/NaCl/ExpandI64.cpp b/lib/Transforms/NaCl/ExpandI64.cpp
index 271c534686..9febcd0ce4 100644
--- a/lib/Transforms/NaCl/ExpandI64.cpp
+++ b/lib/Transforms/NaCl/ExpandI64.cpp
@@ -224,7 +224,7 @@ void ExpandI64::splitInst(Instruction *I, DataLayout& DL) {
       if (T->getIntegerBitWidth() < 32) {
         Low = CopyDebug(new SExtInst(Input, i32, "", I), I);
       } else {
-        Low = Input;
+        Low = CopyDebug(BinaryOperator::Create(Instruction::Or, Input, Zero, "", I), I); // copy the input, hackishly XXX
       }
       Instruction *Check = CopyDebug(new ICmpInst(I, ICmpInst::ICMP_SLT, Low, Zero), I);
       Instruction *High  = CopyDebug(SelectInst::Create(Check, Ones, Zero, "", I), I);
@@ -240,7 +240,7 @@ void ExpandI64::splitInst(Instruction *I, DataLayout& DL) {
       if (T->getIntegerBitWidth() < 32) {
         Low = CopyDebug(new ZExtInst(Input, i32, "", I), I);
       } else {
-        Low = Input;
+        Low = CopyDebug(BinaryOperator::Create(Instruction::Or, Input, Zero, "", I), I); // copy the input, hackishly XXX
       }
       SplitInfo &Split = Splits[I];
       Split.LowHigh.Low = Low;