[PATCH] D70340: Add a key method to Sema to optimize debug info size

[Reid Kleckner via Phabricator via cfe-commits]

rnk added a comment.

In D70340#1748712 <https://reviews.llvm.org/D70340#1748712>, @thakis wrote:

> I don't see any reason not to do this. What's there to discuss? I'm probably missing something obvious.


I guess I was thinking about enabling this only in +asserts builds, so we pay zero overhead in release builds. I was also thinking that if we do implement the "constructor is key for class debug info" flag in the near term, this becomes obsolete. But it's not that much code churn, and it reduces DWARF size with GCC. I guess we could land it after all. :)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70340