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

[Nico Weber via Phabricator via cfe-commits]

thakis added a comment.

In D70340#1748975 <https://reviews.llvm.org/D70340#1748975>, @rnk wrote:

> In D70340#1748712 <https://reviews.llvm.org/D70340#1748712>, @
>
> >
>
>
> 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. :)


With the overhead being the cost of a single vtable with one entry? Or is there more?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70340