[PATCH] D53596: [ThinLTO] Fix a crash in lazy loading of Metadata

Teresa Johnson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 14 12:15:05 PST 2018


tejohnson added a comment.

In https://reviews.llvm.org/D53596#1298860, @steven_wu wrote:

> > Steven - you had a suggestion around callers to getMDNodeFwdRefOrNull, but after this change there will be no more callers to that. So should I go ahead and submit this one, then remove that interface completely in a follow up? Or do it in this patch?
>
> Doesn't matter. Follow up is perfectly fine.


Ok, retesting this one at HEAD, then will remove that interface in a follow on NFC change.


Repository:
  rL LLVM

https://reviews.llvm.org/D53596





More information about the llvm-commits mailing list