[llvm] r263462 - [AliasSetTracker] Do not strip pointer casts when processing MemSetInst

[Michael Kuperstein via llvm-commits]

Author: mkuper
Date: Mon Mar 14 13:34:29 2016
New Revision: 263462

URL: http://llvm.org/viewvc/llvm-project?rev=263462&view=rev
Log:
[AliasSetTracker] Do not strip pointer casts when processing MemSetInst

This fixes PR26843.

Added:
    llvm/trunk/test/Transforms/LICM/pr26843.ll
Modified:
    llvm/trunk/lib/Analysis/AliasSetTracker.cpp

Modified: llvm/trunk/lib/Analysis/AliasSetTracker.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Analysis/AliasSetTracker.cpp?rev=263462&r1=263461&r2=263462&view=diff
==============================================================================
--- llvm/trunk/lib/Analysis/AliasSetTracker.cpp (original)
+++ llvm/trunk/lib/Analysis/AliasSetTracker.cpp Mon Mar 14 13:34:29 2016
@@ -355,7 +355,7 @@ bool AliasSetTracker::add(MemSetInst *MS
     Len = MemoryLocation::UnknownSize;
 
   AliasSet &AS =
-      addPointer(MSI->getDest(), Len, AAInfo, AliasSet::ModAccess, NewPtr);
+      addPointer(MSI->getRawDest(), Len, AAInfo, AliasSet::ModAccess, NewPtr);
   if (MSI->isVolatile())
     AS.setVolatile();
   return NewPtr;
@@ -510,7 +510,7 @@ bool AliasSetTracker::remove(MemSetInst
   else
     Len = MemoryLocation::UnknownSize;
 
-  AliasSet *AS = findAliasSetForPointer(MSI->getDest(), Len, AAInfo);
+  AliasSet *AS = findAliasSetForPointer(MSI->getRawDest(), Len, AAInfo);
   if (!AS)
     return false;
   remove(*AS);

Added: llvm/trunk/test/Transforms/LICM/pr26843.ll
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Transforms/LICM/pr26843.ll?rev=263462&view=auto
==============================================================================
--- llvm/trunk/test/Transforms/LICM/pr26843.ll (added)
+++ llvm/trunk/test/Transforms/LICM/pr26843.ll Mon Mar 14 13:34:29 2016
@@ -0,0 +1,32 @@
+; RUN: opt -S -basicaa -licm < %s | FileCheck %s
+
+target datalayout = "e-m:x-p:32:32-i64:64-f80:32-n8:16:32-a:0:32-S32"
+target triple = "i686-pc-windows-msvc18.0.0"
+
+ at v = common global i32 zeroinitializer, align 4 
+
+; Make sure the store to v is not sunk past the memset
+; CHECK-LABEL: @main
+; CHECK: for.body:
+; CHECK-NEXT: store i32 1, i32* @v
+; CHECK-NEXT: tail call void @llvm.memset
+; CHECK: end:
+; CHECK-NEXT: ret i32 0
+
+define i32 @main(i1 %k) {
+entry:
+  br label %for.body
+ 
+for.body:
+  store i32 1, i32* @v, align 4
+  tail call void @llvm.memset.p0i8.i32(i8* bitcast (i32* @v to i8*), i8 0, i32 4, i32 4, i1 false)
+  br label %for.latch
+  
+for.latch:
+  br i1 %k, label %for.body, label %end
+
+end:
+  ret i32 0
+}
+
+declare void @llvm.memset.p0i8.i32(i8* nocapture, i8, i32, i32, i1)