[PATCH] D116774: AST: Move __va_list tag to the top level on ARM architectures.

[Peter Collingbourne via Phabricator via cfe-commits]

pcc added a comment.

I'm not aware of any of those places causing an actual problem though? The AST isn't a stable interface, and __builtin_dump_struct is for debugging purposes only. I would expect debug info consumers to be able to handle __va_list in the global namespace as this is the status quo for C.

So I'm somewhat inclined to do the simple thing here first, and then look at making things more conditional if a problem comes up.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D116774