[PATCH] D81198: [docs] Specify rules for updating debug locations

[Vedant Kumar via Phabricator via llvm-commits]

vsk added a comment.

In D81198#2101298 <https://reviews.llvm.org/D81198#2101298>, @probinson wrote:

> We recently ran into a case where it looks like the variable's location was correct but the containing scope was wrong.  I guess we'll need a separate update for how to manage scopes.
>  Not an objection to this patch, just observing there are more cases to think about.


Thanks for flagging the issue, it's not something I've spent much time thinking about yet. My plan is to try and write some guidelines for updating debug values next. If you (or anyone else) would like to write up some guidance about dealing with scopes, that would be great!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D81198