[PATCH] D106393: [PowerPC][AIX] Add support for varargs for complex types on AIX
Sean Fertile via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Sep 15 13:07:01 PDT 2021
sfertile accepted this revision as: sfertile.
sfertile added a comment.
This revision is now accepted and ready to land.
LGTM.
================
Comment at: clang/test/CodeGen/aix32-complex-varargs.c:2
+// REQUIRES: powerpc-registered-target
+// RUN: %clang_cc1 -triple powerpc-ibm-aix-xcoff -emit-llvm -o - %s | FileCheck %s
+
----------------
ZarkoCA wrote:
> ZarkoCA wrote:
> > sfertile wrote:
> > > The code-gen for int and float won't change with this patch, lets pre-commit this test without the _Complex short and _Complex char portions now as an NFC patch.
> > I can't precommit this test unless I also remove the fatal error here:
> > ```
> > if (Ty->isAnyComplexType())
> > llvm::report_fatal_error("complex type is not supported on AIX yet");
> > ```
> >
> > But I believe that I can commit the llc tests and I'll go ahead and do that.
> Sorry, not commit the llc tests but separate them in their own patch.
Sorry, I missed that. I was thinking since the word-size element types didn't change we should recommit heir tests, but since it was disabled with a fatal error that doesn't make sense.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D106393/new/
https://reviews.llvm.org/D106393
More information about the cfe-commits
mailing list