[PATCH] D14697: [Verifier] Fix !dbg validation if Scope is the Subprogram

Keno Fischer via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 16 15:28:43 PST 2015


loladiro added a comment.

I'm not sure the added complexity of walking the chain and checking everything would be worth it. It seems like that would just be manually inlining  getInlinedAtScope + doing the memoization. It seems like a reasonable assumption that the scope declared on one instruction would also be declared on instructions nearby, while that's not necessarily the case for intermediate ones (plus if it is we'll put it in the list when we get there).


http://reviews.llvm.org/D14697





More information about the llvm-commits mailing list