diff options
| author | Michael Platings <michael.platings@arm.com> | 2023-07-07 09:37:34 +0100 |
|---|---|---|
| committer | Michael Platings <michael.platings@arm.com> | 2023-07-07 09:41:18 +0100 |
| commit | 79165735e11e40e1fbbb91ccbdd4df24d6349837 (patch) | |
| tree | 33bd5000b8c4235fa943b49302418efd07a98eef /clang/lib/Frontend/CompilerInvocation.cpp | |
| parent | 336d7281ad61fbd4a14fa2ef3d2bde5a63d10417 (diff) | |
| download | llvm-79165735e11e40e1fbbb91ccbdd4df24d6349837.zip llvm-79165735e11e40e1fbbb91ccbdd4df24d6349837.tar.gz llvm-79165735e11e40e1fbbb91ccbdd4df24d6349837.tar.bz2 | |
[ARM][Driver] Change float-abi warning
Previously the warning stated "flag ignored" which is only partially
true - the invalid flag would prevent -feature +soft-float-abi from
being emitted which resulted in user-visible behaviour like
__ARM_PCS_VFP being defined.
Rather than attempt to coerce invalid flags into valid behaviour, don't
describe the expected behaviour.
Ideally the warning would be an error, as it is in GCC. However there
are tests in llvm-project that trigger the warning. Therefore one has to
assume that making the warning an error would break other code that
already exists in the wild.
Also apply test improvements suggested by @MaskRay on D150902.
Reviewed By: simon_tatham
Differential Revision: https://reviews.llvm.org/D154578
Diffstat (limited to 'clang/lib/Frontend/CompilerInvocation.cpp')
0 files changed, 0 insertions, 0 deletions