commit 30ae859c696748e1314788367226e4c65669f526
author Johannes Doerfert <johannes@jdoerfert.de> Fri Jan 10 12:13:10 2020 -0600
committer Johannes Doerfert <johannes@jdoerfert.de> Thu Jan 23 18:13:52 2020 -0600
tree 5057e6de8d4411affdc3d0d08c8d81a4704fe15f
parent 59733525d37cf9ad88b5021b33ecdbaf2e18911c

[Attributor][FIX] Store alignment only holds for the pointer value

We accidentally used the store alignment for the value operand as well,
which is incorrect and crashed the SPASS application in the test suite.

llvm/lib/Transforms/IPO/Attributor.cpp

diff --git a/llvm/lib/Transforms/IPO/Attributor.cpp b/llvm/lib/Transforms/IPO/Attributor.cpp
index baa1cb1..5eb21a0 100644
--- a/llvm/lib/Transforms/IPO/Attributor.cpp
+++ b/llvm/lib/Transforms/IPO/Attributor.cpp
@@ -3420,9 +3420,10 @@
   }
 
   const Value *UseV = U->get();
-  if (auto *SI = dyn_cast<StoreInst>(I))
-    Alignment = SI->getAlignment();
-  else if (auto *LI = dyn_cast<LoadInst>(I))
+  if (auto *SI = dyn_cast<StoreInst>(I)) {
+    if (SI->getPointerOperand() == UseV)
+      Alignment = SI->getAlignment();
+  } else if (auto *LI = dyn_cast<LoadInst>(I))
     Alignment = LI->getAlignment();
 
   if (Alignment <= 1)