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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to