[PATCH] D18634: Don't IPO over functions that can be de-refined

Mehdi AMINI via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 30 17:30:35 PDT 2016


joker.eph added inline comments.

================
Comment at: include/llvm/IR/GlobalValue.h:300
@@ -263,1 +299,3 @@
+  }
+
   bool hasExternalLinkage() const { return isExternalLinkage(getLinkage()); }
----------------
Isn't it what `isStrongDefinitionForLinker()` would return?

I don't see why we need a new predicate? (there are probably already too many...)


http://reviews.llvm.org/D18634





More information about the llvm-commits mailing list