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

Peter Collingbourne via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jun 23 18:21:35 PDT 2021


pcc created this revision.
pcc added reviewers: rsmith, eugenis.
pcc requested review of this revision.
Herald added a project: clang.

This ensures that the mangled type names match between C and C++,
which is significant when using -fsanitize=cfi-icall. Ideally we
wouldn't have created this namespace at all, but it's now part of
the ABI (e.g. in mangled names), so we can't change it.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D104830

Files:
  clang/lib/AST/ASTContext.cpp
  clang/test/CodeGen/aarch64-varargs.c
  clang/test/CodeGen/arm64-be-hfa-vararg.c
  clang/test/Headers/stdarg.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D104830.354129.patch
Type: text/x-patch
Size: 69645 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20210624/eb14fe50/attachment-0001.bin>


More information about the cfe-commits mailing list