On Thu, Mar 16, 2017 at 5:45 PM Hal Finkel <hfin...@anl.gov> wrote: > > On 03/16/2017 07:40 PM, Eric Christopher wrote: > > > > On Thu, Mar 16, 2017 at 5:37 PM Hal Finkel via Phabricator < > revi...@reviews.llvm.org> wrote: > > hfinkel added a comment. > > In https://reviews.llvm.org/D30415#703398, @echristo wrote: > > > Different suggestion: > > > > Remove the faltivec option. Even gcc doesn't support it anymore afaict. > > > What are you suggesting? Always having the language extensions on? Or > explicitly tying the language extensions to the underlying target feature? > > > I was thinking the latter. > > > Is that what GCC now does? > > That would be my guess given the option isn't listed anymore, but what it does is this:
echristo@dzur ~/s/gcc-git> grep -r faltivec * gcc/testsuite/gcc.target/powerpc/stabs-attrib-vect-darwin.c:/* { dg-options "-gstabs+ -fno-eliminate-unused-debug-types -faltivec" } */ gcc/testsuite/ChangeLog-1993-2007: * g++.dg/ext/altivec-8.C: Use '-maltivec' instead of '-faltivec'; gcc/config/rs6000/darwin.h: the kernel or some such. The "-faltivec" option should have been gcc/config/rs6000/darwin.h: %{faltivec:-maltivec -include altivec.h} %{fno-altivec:-mno-altivec} \ gcc/config/rs6000/darwin.h: %<faltivec %<fno-altivec " \ gcc/config/rs6000/darwin.opt:faltivec gcc/ChangeLog-2003: -maltivec is specified, not the non-existent -faltivec. gcc/ChangeLog-2010: * config/rs6000/darwin.h (CC1_SPEC): Handle -faltivec and -fno-altivec. gcc/ChangeLog-2010: * config/rs6000/darwin.opt (Waltivec-long-deprecated, faltivec, and only on darwin. I don't see anything that treats the faltivec alias as anything language specific anywhere. It basically just says "pass the include and turn on maltivec". At this point I'm pretty sure that -faltivec can just be ignored. -eric > -Hal > > > > -eric > > > > (Go ahead and commit the zvector part if you'd like). > > > > -eric > > > > > https://reviews.llvm.org/D30415 > > > > > -- > Hal Finkel > Lead, Compiler Technology and Programming Languages > Leadership Computing Facility > Argonne National Laboratory > >
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits