[PATCH] D116774: AST: Move __va_list tag back to std conditionally on AArch64.

Jessica Clarke via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Feb 17 11:18:35 PST 2022


jrtc27 added inline comments.


================
Comment at: clang/lib/AST/ASTContext.cpp:8555-8556
+    // namespace std { struct __va_list {
+    NamespaceDecl *NS;
+    NS = NamespaceDecl::Create(const_cast<ASTContext &>(*Context),
+                               Context->getTranslationUnitDecl(),
----------------
aaron.ballman wrote:
> 
Hm, the advantage of leaving it as it is is then it completely reverts D104830 (ignoring the added CFI test) rather than being written in a slightly different but equivalent way. Don't really care either way myself though, both have their merits.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D116774



More information about the cfe-commits mailing list