[PATCH] D21513: [CFLAA] Try to be less conservative on more functions

Sanjoy Das via llvm-commits llvm-commits at lists.llvm.org
Sun Jun 19 22:44:10 PDT 2016


sanjoy added a subscriber: sanjoy.
sanjoy added a comment.

Drop by comment


================
Comment at: lib/Analysis/CFLAliasAnalysis.cpp:371
@@ -370,3 +370,3 @@
   static bool isFunctionExternal(Function *Fn) {
-    return Fn->isDeclaration() || !Fn->hasLocalLinkage();
+    return Fn->isDeclaration() || Fn->isInterposable();
   }
----------------
If this is doing inter-procedural alias analysis, then you need to use `Fn-> hasExactDefinition()`.


http://reviews.llvm.org/D21513





More information about the llvm-commits mailing list