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

Reply via email to