[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