[PATCH] D118904: [clang][CodeGen] Use memory type representation in `va_arg`

Jan Svoboda via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Feb 3 07:04:36 PST 2022


jansvoboda11 added a comment.

Should we add tests for other targets besides arm64?



================
Comment at: clang/test/CodeGen/arm64-arguments.c:203
+  __builtin_va_start(ap, i);
+  // TODO: Add proper checks here.
+  _Bool b = __builtin_va_arg(ap, _Bool);
----------------
Not sure what to check here. Currently, we're just verifying the compiler doesn't crash.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D118904



More information about the cfe-commits mailing list