[ARM] Fix recent breakage of -mfpu=none.
authorSimon Tatham <simon.tatham@arm.com>
Mon, 3 Jun 2019 11:02:53 +0000 (11:02 +0000)
committerSimon Tatham <simon.tatham@arm.com>
Mon, 3 Jun 2019 11:02:53 +0000 (11:02 +0000)
commitdc83a3c44940ca4d13839613dc55049e5eddc42e
tree20dd4fabc8599c75138ac84dabe11f587dd610d4
parentd8d3e17b8b07a34e6118a8b6cd66aea657265531
[ARM] Fix recent breakage of -mfpu=none.

The recent change D60691 introduced a bug in clang when handling
option combinations such as `-mcpu=cortex-m4 -mfpu=none`. Those
options together should select Cortex-M4 but disable all use of
hardware FP, but in fact, now hardware FP instructions can still be
generated in that mode.

The reason is because the handling of FPUVersion::NONE disables all
the same feature names it used to, of which the base one is `vfp2`.
But now there are further features below that, like `vfp2d16fp` and
(following D60694) `fpregs`, which also need to be turned off to
disable hardware FP completely.

Added a tiny test which double-checks that compiling a simple FP
function doesn't access the FP registers.

Reviewers: SjoerdMeijer, dmgreen

Reviewed By: dmgreen

Subscribers: lebedev.ri, javed.absar, kristof.beyls, hiraditya, cfe-commits, llvm-commits

Tags: #clang, #llvm

Differential Revision: https://reviews.llvm.org/D62729

llvm-svn: 362380
clang/lib/Driver/ToolChains/Arch/ARM.cpp
clang/test/CodeGen/arm-mfpu-none.c [new file with mode: 0644]
clang/test/Driver/arm-mfpu.c
llvm/lib/Support/ARMTargetParser.cpp