Remove unnecessary pointer casts on store instructions.

BUG= https://code.google.com/p/nativeclient/issues/detail?id=3544
R=mseaborn@chromium.org

Review URL: https://codereview.chromium.org/21964002

1 files changed, 5 insertions, 0 deletions

diff --git a/lib/Bitcode/NaCl/Writer/NaClValueEnumerator.cpp b/lib/Bitcode/NaCl/Writer/NaClValueEnumerator.cpp
index 11e1161bf8..34f4f2bbe9 100644
--- a/lib/Bitcode/NaCl/Writer/NaClValueEnumerator.cpp
+++ b/lib/Bitcode/NaCl/Writer/NaClValueEnumerator.cpp
@@ -443,7 +443,12 @@ static bool AllowsNormalizedPtr(const Value *V, const Instruction *Arg) {
   default:
     return false;
   case Instruction::Load:
+    // Verify it is the ptr argument of the load.  Note: This check is
+    // not really necessary in that a load only has one argument.
     return I->getOperand(0) == Arg;
+  case Instruction::Store:
+    // Verify it is the ptr argument of the store.
+    return I->getOperand(1) == Arg;
   }
 }