Always create a shadow node for a store instruction, even if it's storing

a scalar value.  Likewise for load instructions.

llvm-svn: 2990

1 files changed, 3 insertions, 3 deletions

diff --git a/llvm/lib/Analysis/DataStructure/Local.cpp b/llvm/lib/Analysis/DataStructure/Local.cpp
index 6d8b1b385dc..6ce3ae08580 100644
--- a/llvm/lib/Analysis/DataStructure/Local.cpp
+++ b/llvm/lib/Analysis/DataStructure/Local.cpp
@@ -256,16 +256,16 @@ void GraphBuilder::visitGetElementPtrInst(GetElementPtrInst &GEP) {
 }
 
 void GraphBuilder::visitLoadInst(LoadInst &LI) {
-  if (!isa<PointerType>(LI.getType())) return; // Only pointer PHIs
   DSNode *Ptr = getSubscriptedNode(LI, getValueNode(*LI.getOperand(0)));
+  if (!isa<PointerType>(LI.getType())) return; // Only pointer PHIs
   getValueNode(LI)->addEdgeTo(getLink(Ptr, 0));
 }
 
 void GraphBuilder::visitStoreInst(StoreInst &SI) {
+  DSNode *DestPtr = getSubscriptedNode(SI, getValueNode(*SI.getOperand(1)));
   if (!isa<PointerType>(SI.getOperand(0)->getType())) return;
   DSNode *Value   = getValueNode(*SI.getOperand(0));
-  DSNode *DestPtr = getValueNode(*SI.getOperand(1));
-  getSubscriptedNode(SI, DestPtr)->addEdgeTo(getLink(Value, 0));
+  DestPtr->addEdgeTo(getLink(Value, 0));
 }
 
 void GraphBuilder::visitReturnInst(ReturnInst &RI) {