filcab added a comment.

Hi @hctim, thanks for the patch.
I have one question, though. Do you really need to remove the information you 
removed?
Some people might be testing ASan binaries without access to debug info 
(because it's been stripped or it's not been loaded and is not available for 
the sanitizers to get symbols from, etc), and having the full global 
information would be useful for those reports.

Can this be done by having a flag to make clang not emit the source information 
+ the sanitizer patch to get the line and column?
That way we could keep the current behaviour of emitting more info, and you'd 
still get your savings + use of debuginfo.

Thank you,
Filipe


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D124493

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

Reply via email to