[PATCH] D75213: RFC: More principled implementation of DISubprogram::describes()

Fri Feb 28 14:00:28 PST 2020

aprantl added a comment.

In D75213#1898741 <https://reviews.llvm.org/D75213#1898741>, @dexonsmith wrote:

> Okay, LGTM then, as long as...
>
> In D75213#1896981 <https://reviews.llvm.org/D75213#1896981>, @aprantl wrote:
>
> > ThinLTO and FullLTO stage builds of all dependencies of check-clang succeeded with this change.
>
>
> Did you confirm that none of the debug info was stripped?  IIRC, if debug info doesn't verify during LTO it gets stripped rather than causing a failure.

Good point! I should have seen that as a warning in the linker output if it happened.

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D75213/new/

https://reviews.llvm.org/D75213