[PATCH] D37518: Fix PR33878: BasicAA incorrectly assumes different address spaces don't alias

[Nuno Lopes via Phabricator via llvm-commits]

This revision was automatically updated to reflect the committed changes.
Closed by commit rL312648: Fix PR33878: BasicAA incorrectly assumes different address spaces don't alias (authored by nlopes).

Changed prior to commit:
  https://reviews.llvm.org/D37518?vs=114009&id=114024#toc

Repository:
  rL LLVM

https://reviews.llvm.org/D37518

Files:
  llvm/trunk/lib/Analysis/BasicAliasAnalysis.cpp


Index: llvm/trunk/lib/Analysis/BasicAliasAnalysis.cpp
===================================================================
--- llvm/trunk/lib/Analysis/BasicAliasAnalysis.cpp
+++ llvm/trunk/lib/Analysis/BasicAliasAnalysis.cpp
@@ -1590,11 +1590,6 @@
         (isa<Argument>(O2) && isIdentifiedFunctionLocal(O1)))
       return NoAlias;
 
-    // Most objects can't alias null.
-    if ((isa<ConstantPointerNull>(O2) && isKnownNonNull(O1)) ||
-        (isa<ConstantPointerNull>(O1) && isKnownNonNull(O2)))
-      return NoAlias;
-
     // If one pointer is the result of a call/invoke or load and the other is a
     // non-escaping local object within the same function, then we know the
     // object couldn't escape to a point where the call could return it.


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D37518.114024.patch
Type: text/x-patch
Size: 759 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170906/502fd6eb/attachment.bin>