[PATCH] D17867: [gold] Handle modules that are not included in the link.

Evgeniy Stepanov via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 3 14:53:08 PST 2016


eugenis added inline comments.

================
Comment at: tools/gold/gold-plugin.cpp:58
@@ -57,1 +57,3 @@
 
+#define LDPT_GET_SYMBOLS_V3 28
+
----------------
rafael wrote:
> you need a ifndef, no?
It's an enum, not a macro. The samewith LDPO_PIE above, we could remove the #ifdef - unless it's meant to be configurable with -DLDPO_PIE or something (but that seems unlikely).



Repository:
  rL LLVM

http://reviews.llvm.org/D17867





More information about the llvm-commits mailing list