]> granicus.if.org Git - clang/commit
[ARM] Replace fp-only-sp and d16 with fp64 and d32.
authorSimon Tatham <simon.tatham@arm.com>
Tue, 28 May 2019 16:13:20 +0000 (16:13 +0000)
committerSimon Tatham <simon.tatham@arm.com>
Tue, 28 May 2019 16:13:20 +0000 (16:13 +0000)
commit173708d165c6145075ef941e6953f2c1c1167544
treef7df6a901f15b2c070c4c9ad602991ec351fbc15
parentadbb459ee00ba8b946135bf445109dd34b81d614
[ARM] Replace fp-only-sp and d16 with fp64 and d32.

Those two subtarget features were awkward because their semantics are
reversed: each one indicates the _lack_ of support for something in
the architecture, rather than the presence. As a consequence, you
don't get the behavior you want if you combine two sets of feature
bits.

Each SubtargetFeature for an FP architecture version now comes in four
versions, one for each combination of those options. So you can still
say (for example) '+vfp2' in a feature string and it will mean what
it's always meant, but there's a new string '+vfp2d16sp' meaning the
version without those extra options.

A lot of this change is just mechanically replacing positive checks
for the old features with negative checks for the new ones. But one
more interesting change is that I've rearranged getFPUFeatures() so
that the main FPU feature is appended to the output list *before*
rather than after the features derived from the Restriction field, so
that -fp64 and -d32 can override defaults added by the main feature.

Reviewers: dmgreen, samparker, SjoerdMeijer

Subscribers: srhines, javed.absar, eraman, kristof.beyls, hiraditya, zzheng, Petar.Avramovic, cfe-commits, llvm-commits

Tags: #clang, #llvm

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

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@361845 91177308-0d34-0410-b5e6-96231b3b80d8
lib/Basic/Targets/ARM.cpp
test/CodeGen/arm-target-features.c
test/Driver/arm-mfpu.c