[PATCH] D15351: Change hasInitializer() to call isStrongDefinitionForLinker() instead of !isWeakForLinker()

Peter Collingbourne via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 8 16:52:46 PST 2015


pcc added inline comments.

================
Comment at: include/llvm/IR/GlobalVariable.h:108
@@ -107,2 +107,3 @@
   inline bool hasUniqueInitializer() const {
     return hasInitializer() &&
+      // We need to be sure this is the definition that will actually be used
----------------
The call to `hasInitializer()` is redundant now.


http://reviews.llvm.org/D15351





More information about the llvm-commits mailing list