[PATCH] D75175: [CallSiteInfo] Enable the call site info only for -g + optimizations

Thu Feb 27 07:56:33 PST 2020

djtodoro added a comment.

As I mentioned on D74904 <https://reviews.llvm.org/D74904>, I think it makes sense to avoiding production of call site info in the case of non-debug builds, since it is being used only for Debug Entry Values feature.
The Call Site Info potentially could be used by some other LLVM modules, but if we find such case, we can enable it for that purpose as well.

The separate option makes sense to me, since the Call Site Info is conceptually different thing, and the Debug Entry Values feature just uses the Call Site Info utility.

If this seems reasonable, I will rebase the re-landing of D73534 <https://reviews.llvm.org/D73534> on top of this.

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

https://reviews.llvm.org/D75175