[PATCH] D53890: [LTO] Record LTOUnit flag in index and validate during LTO link

[Teresa Johnson via Phabricator via llvm-commits]

tejohnson added a comment.

In https://reviews.llvm.org/D53890#1293818, @pcc wrote:

> > without LTOUnit (for purely summary-based class hierarchy analyses).
>
> We might want a different name for the flag then, because a translation unit that is subject to purely summary-based class hierarchy analysis is still part of the LTO unit, it just participates in fewer optimizations.


I think I am confused then about the terminology. My understanding was that the LTO unit was the part that would be linked with regular LTO. Does a linkage unit's LTO unit include both the regular and thin LTO modules? If so, what name would you suggest?


Repository:
  rL LLVM

https://reviews.llvm.org/D53890