Some types (e.g. `_Bool`) have different scalar and memory representations. CodeGen for `va_arg` didn't take this into account, leading to an assertion failures with different types.
This patch makes sure we use memory representation for `va_arg`.
Reviewed By: ahatanak
Differential Revision: https://reviews.llvm.org/D118904
"Unexpected CoerceToType seen in arginfo in generic VAArg emitter!");
Address Temp = CGF.CreateMemTemp(Ty, "varet");
- Val = CGF.Builder.CreateVAArg(VAListAddr.getPointer(), CGF.ConvertType(Ty));
+ Val = CGF.Builder.CreateVAArg(VAListAddr.getPointer(),
+ CGF.ConvertTypeForMem(Ty));
CGF.Builder.CreateStore(Val, Temp);
return Temp;
}
__builtin_va_end(ap);
return ll;
}
+_Bool t3(int i, ...) {
+ // CHECK: t3
+ __builtin_va_list ap;
+ __builtin_va_start(ap, i);
+ // CHECK: %0 = va_arg {{.*}}* %ap, i8
+ // CHECK-NEXT: store i8 %0, i8* %varet, align 1
+ _Bool b = __builtin_va_arg(ap, _Bool);
+ __builtin_va_end(ap);
+ return b;
+}
#include <arm_neon.h>