[PATCH] D134825: [clang] Add debug info in MicrosoftCXXABI::EmitVirtualMemPtrThunk()

[David Blaikie via Phabricator via cfe-commits]

dblaikie accepted this revision.
dblaikie added a comment.
This revision is now accepted and ready to land.

Sounds good to me.

(Ah, I see, that chrome bug actually hit the underlying problem, not even the verifier constraint - when the constraint is violated & not detected, then code gets inlined without being stitched up to the inline scope and you end up with instructions in a function where the function has one subprogram, but the scope chain of the instruction's location leads to a different subprogram - which is a problem/makes building the scope tree weird/not possible (because it's no longer a tree but a forest))


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D134825