[PATCH] D76130: [PPC][AIX] Implement variadic function handling in LowerFormalArguments_AIX

Jason Liu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Mar 18 13:35:24 PDT 2020


jasonliu added inline comments.


================
Comment at: llvm/test/CodeGen/PowerPC/aix-cc-abi-va_args-32.ll:7
+; RUN: FileCheck --check-prefixes=CHECKASM,ASM32PWR4 %s
+
+declare void @llvm.lifetime.start.p0i8(i64 immarg, i8* nocapture)
----------------
A general comment about how to test va_arg.
I think the two test cases we have here are not enough. 
On top of my head I would probably add in getting double as va_arg argument, uses of va_copy and so on. 
It's probably worth checking how other targets test va_arg. Grep "va_start" in llvm/test/CodeGen would give you a lot of hit. 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76130





More information about the cfe-commits mailing list