[PATCH] D104830: AST: Create __va_list in the std namespace even in C.

Jessica Clarke via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Nov 13 18:27:22 PST 2021


jrtc27 added a comment.

This results in the DWARF type info containing DW_TAG_namespace for C, which breaks DTrace's CTF (as the name implies, it is for C, not C++). This does not seem correct to me. If your mangling for va_list is broken then you should special case that in the CFI mangler as an ABI quirk, IMO, not change the DWARF for C to contain C++ things.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D104830



More information about the cfe-commits mailing list