[llvm-commits] [PATCH, test-suite, AltiVec][2/4] Fix pre-ANSI C

Ulrich Weigand Ulrich.Weigand at de.ibm.com
Fri Oct 12 08:30:32 PDT 2012


Hal Finkel <hfinkel at anl.gov> wrote on 12.10.2012 15:31:40:

> I think this is fine.

Thanks for the review; I've now checked all four patches in.

> Out of curiosity, do all relevant compilers
> (that understand AliVec) also understand __complex__, or should we
> change to the C99 _Complex keyword)?

There's at least one other test that uses __complex__
(SingleSource/Benchmarks/Misc/mandel.c), so I'd expect
this to be supported by all relevant compilers ...

But I guess we could as well change to _Complex -- would
you like me to prepare a patch to do so?

Bye,
Ulrich




More information about the llvm-commits mailing list