echristo added a comment. In https://reviews.llvm.org/D30415#705196, @uweigand wrote:
> In https://reviews.llvm.org/D30415#704761, @echristo wrote: > > > In https://reviews.llvm.org/D30415#703652, @uweigand wrote: > > > > > I'm a bit confused by this discussion. -faltivec and -maltivec are > > > simply aliases, they do exactly the same thing; the clang-internal > > > variable OPT_faltivec indicates the use of either -faltivec or -maltivec. > > > > > > They didn't used to, I arranged it so that they did (technically breaking > > gcc compatibility) a while ago. > > > Well, mainline GCC doesn't have -faltivec at all and never had, I think this > was only an Apple GCC extension ... Not sure what exactly the semantics of > that was. Sure it does and has for years. Check out rs6000/darwin.h :) FWIW: It turns on maltivec and adds a -include of altivec.h >>> Or is the suggestion to simply remove the alias -faltivec, and leave >>> -maltivec as-is? I'd be less opposed to this since it probably breaks >>> fewer users ... but I'm still not quite sure what it actually buys us. >>> And in any case the patch currently under discussion here would still be >>> necessary then, to fix -maltivec -mno-altivec ... >> >> No, remove faltivec and move forward with -maltivec/-mno-altivec but you >> should be able to remove a lot of the special handling at that point. > > I'm still confused as to what exactly you're refering to here. As far as I > can see, every single thing triggered by -faltivec / -maltivec in the > compiler frontend would still be needed exactly the same if we only supported > the -maltivec option name. So the only thing we'd save is literally the two > lines in include/clang/Driver/Options.td that set up the alias. > > Do you have an example of the "special handling" to remove you're thinking of? Nearly all of the code in lib/Driver/ToolChains/Clang.cpp and lib/Driver/ToolChains/Arch/PPC.cpp that deal with altivec. Simplifying the interface by getting rid of needing to check multiple options. -eric https://reviews.llvm.org/D30415 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits