diff options
| author | Dan Albert <danalbert@google.com> | 2019-02-15 20:31:54 +0000 |
|---|---|---|
| committer | Dan Albert <danalbert@google.com> | 2019-02-15 20:31:54 +0000 |
| commit | df89c6ffa37366a07d2e3a07ce3985a3b8bf754e (patch) | |
| tree | ce855e1373b22c701910c574cc0e762fdaeb0a04 /clang/lib/Driver/ToolChains/Arch/ARM.cpp | |
| parent | e308e4299d03e13c59d57a8cb3ed4ebbb7d710b2 (diff) | |
| download | bcm5719-llvm-df89c6ffa37366a07d2e3a07ce3985a3b8bf754e.tar.gz bcm5719-llvm-df89c6ffa37366a07d2e3a07ce3985a3b8bf754e.zip | |
[Driver] Default all Android ARM targets to NEON.
Summary:
There are an insignificant number of ARM Android devices that don't
support NEON. Default to using NEON since that will improve
performance on the majority of devices. Users that need to target
non-NEON devices can still explicitly disable NEON.
Reviewers: srhines, pirama, kristof.beyls
Reviewed By: pirama
Subscribers: efriedma, javed.absar, cfe-commits
Tags: #clang
Differential Revision: https://reviews.llvm.org/D58153
llvm-svn: 354166
Diffstat (limited to 'clang/lib/Driver/ToolChains/Arch/ARM.cpp')
| -rw-r--r-- | clang/lib/Driver/ToolChains/Arch/ARM.cpp | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/clang/lib/Driver/ToolChains/Arch/ARM.cpp b/clang/lib/Driver/ToolChains/Arch/ARM.cpp index 1a6636ac329..c641e5c1711 100644 --- a/clang/lib/Driver/ToolChains/Arch/ARM.cpp +++ b/clang/lib/Driver/ToolChains/Arch/ARM.cpp @@ -378,9 +378,7 @@ void arm::getARMTargetFeatures(const ToolChain &TC, } else if (FPUArg) { getARMFPUFeatures(D, FPUArg, Args, FPUArg->getValue(), Features); } else if (Triple.isAndroid() && getARMSubArchVersionNumber(Triple) >= 7) { - // Android mandates minimum FPU requirements based on OS version. - const char *AndroidFPU = - Triple.isAndroidVersionLT(23) ? "vfpv3-d16" : "neon"; + const char *AndroidFPU = "neon"; if (!llvm::ARM::getFPUFeatures(llvm::ARM::parseFPU(AndroidFPU), Features)) D.Diag(clang::diag::err_drv_clang_unsupported) << std::string("-mfpu=") + AndroidFPU; |
