[PATCH] D30415: Fix -mno-altivec cannot overwrite -maltivec option

Eric Christopher via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Mar 21 15:19:03 PDT 2017


echristo added a comment.

In https://reviews.llvm.org/D30415#706370, @sfertile wrote:

> > I have a patch to do this now. I'll plan on committing it in a bit.
>
> Is there a way to mark the -f form of the option as deprecated? We should warn and suggest users switch to the -maltivec option for a release to give people a chance to update their builds.


I've added something and committed all of my work here:

echristo at athyra ~/s/l/t/clang> git svn dcommit
Committing to https://llvm.org/svn/llvm-project/cfe/trunk ...
	A	test/Driver/unsupported-faltivec.c
	M	docs/ClangCommandLineReference.rst
	M	docs/LanguageExtensions.rst
	M	include/clang/Basic/DiagnosticDriverKinds.td
	M	include/clang/Driver/Options.td
	M	lib/Basic/Targets.cpp
	M	lib/Driver/ToolChains/Arch/PPC.cpp
	M	lib/Driver/ToolChains/Clang.cpp
	M	lib/Frontend/CompilerInstance.cpp
	M	lib/Frontend/CompilerInvocation.cpp
	M	test/CodeGen/altivec.c
	M	test/CodeGen/builtins-ppc-altivec.c
	M	test/CodeGen/builtins-ppc-crypto-disabled.c
	M	test/CodeGen/builtins-ppc-crypto.c
	M	test/CodeGen/builtins-ppc-error.c
	M	test/CodeGen/builtins-ppc-htm.c
	M	test/CodeGen/builtins-ppc-p8vector.c
	M	test/CodeGen/builtins-ppc-p9vector.c
	M	test/CodeGen/builtins-ppc-quadword.c
	M	test/CodeGen/builtins-ppc-vsx.c
	M	test/CodeGen/ppc64-align-struct.c
	M	test/CodeGen/ppc64-vector.c
	M	test/CodeGen/ppc64le-aggregates.c
	M	test/Driver/ppc-features.cpp
	M	test/Headers/altivec-header.c
	M	test/Headers/altivec-intrin.c
	M	test/Parser/altivec-csk-bool.c
	M	test/Parser/altivec.c
	M	test/Parser/cxx-altivec.cpp
	M	test/Parser/vsx.c
	M	test/Sema/altivec-init.c
	M	test/Sema/builtins-ppc.c
	M	test/SemaCXX/altivec.cpp
	M	test/SemaCXX/cxx-altivec.cpp
Committed r298449


https://reviews.llvm.org/D30415





More information about the cfe-commits mailing list