[PATCH] D20268: [wip] Resolution-based LTO API.

Mehdi AMINI via llvm-commits llvm-commits at lists.llvm.org
Sat May 14 22:20:38 PDT 2016


joker.eph added inline comments.

================
Comment at: include/llvm/LTO/LTO.h:27
@@ +26,3 @@
+  unsigned FinalDefinitionInLinkageUnit : 1;
+
+  /// The definition of this symbol is visible outside of the LTO unit.
----------------
pcc wrote:
> joker.eph wrote:
> > D20217 on declaration would work, but my point was rather that there is no "symbol resolution" here that helps with setting this on decl.
> > 
> That would be `!Prevailing && FinalDefinitionInLinkageUnit`, no?
Oh, right.


http://reviews.llvm.org/D20268





More information about the llvm-commits mailing list