[PATCH] D66352: Debug Info: Support for DW_AT_export_symbols for anonymous structs

Adrian Prantl via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 19 15:57:02 PDT 2019


aprantl added a comment.

In D66352#1636225 <https://reviews.llvm.org/D66352#1636225>, @dblaikie wrote:

> Eh, I think it's still pretty unnecessary (& the DWARF spec is only suggestive, doesn't tend to have requirements in this regard) and so I'd probably suggest waiting until some consumer really wants this (& I'd still want to have a discussion with the consumer about why they find this to be needed). But it's really cheap (especially in DWARFv5 where it can use a const_value form & cost no bytes in debug_info (if/when that sort of thing is implemented in LLVM, if it isn't already).


@shafik, Can you outline the problem Clang has differentiating anonymous structs and lambdas that made this patch necessary?


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

https://reviews.llvm.org/D66352





More information about the llvm-commits mailing list