[PATCH] Correctly combine alias.scope metadata by a union instead of intersecting

hfinkel at anl.gov hfinkel at anl.gov
Sun Feb 8 09:05:39 PST 2015


Ack, you're right. Thanks for catching that. LGTM.


================
Comment at: lib/Analysis/TypeBasedAliasAnalysis.cpp:627
@@ -626,3 +626,3 @@
     N.Scope =
-        MDNode::intersect(N.Scope, getMetadata(LLVMContext::MD_alias_scope));
+        MDNode::getMostGenericAliasScope(N.Scope, getMetadata(LLVMContext::MD_alias_scope));
   else
----------------
Line too long?

http://reviews.llvm.org/D7490

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list