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



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

Reply via email to