ZarkoCA accepted this revision.
ZarkoCA added a comment.

It may be better to use `__builtin_va_list` and the like instead of the header 
inclusion but I don't want to make the test change too onerous if that turns 
out to be the case.



================
Comment at: clang/test/CodeGen/aix-altivec-vaargs.c:5
+
+#include <stdarg.h>
+
----------------
Any reason for for including the header instead of builtins as in aix-vaargs.c? 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D97474

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to