[PATCH] D124493: Move Sanitizer metadata to be on-GlobalValue.

[Evgenii Stepanov via Phabricator via cfe-commits]

eugenis added a comment.

I've always viewed the current implementation as a hack, and believed this data should live in debug info.

It can be convenient to get symbolized reports for global overflow bug in a stripped binary, but those bugs are quite rare, and "symbolization" only affects the global itself, not the stack trace of the bad memory access. It does not seem worth it to support two code paths just for this.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D124493