ZarkoCA added inline comments.
================ 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: > 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. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D106393/new/ https://reviews.llvm.org/D106393 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits