diff options
author | Richard Sandiford <richard.sandiford@arm.com> | 2019-01-15 16:46:54 +0000 |
---|---|---|
committer | Richard Sandiford <rsandifo@gcc.gnu.org> | 2019-01-15 16:46:54 +0000 |
commit | 99063eeecbfb15fece3c72747a9dd14077c41afd (patch) | |
tree | 907872956c50b6376f90ebf8728f92652457d81c /gcc/d | |
parent | 17f781605e695711a3383b0474e806ce8150cf49 (diff) | |
download | gcc-99063eeecbfb15fece3c72747a9dd14077c41afd.zip gcc-99063eeecbfb15fece3c72747a9dd14077c41afd.tar.gz gcc-99063eeecbfb15fece3c72747a9dd14077c41afd.tar.bz2 |
PR inline-asm/52813 revisited
The original patch for this PR made it an error to list the stack
pointer in the clobber list of an inline asm. However, the general
feeling seemed to be that going straight to a hard error was too harsh,
since there's quite a bit of existing code that has the clobber.
This patch implements the compromise discussed on IRC of making it
a -Wdeprecated warning instead.
2019-01-15 Richard Sandiford <richard.sandiford@arm.com>
gcc/
PR inline-asm/52813
* doc/extend.texi: Document that listing the stack pointer in the
clobber list of an asm is a deprecated feature.
* common.opt (Wdeprecated): Moved from c-family/c.opt.
* cfgexpand.c (asm_clobber_reg_is_valid): Issue a -Wdeprecated
warning instead of an error for clobbers of the stack pointer.
Add a note explaining why.
gcc/c-family/
PR inline-asm/52813
* c.opt (Wdeprecated): Move documentation and variable to common.opt.
gcc/d/
PR inline-asm/52813
* lang.opt (Wdeprecated): Reference common.opt instead of c.opt.
gcc/testsuite/
PR inline-asm/52813
* gcc.target/i386/pr52813.c (test1): Turn the diagnostic into a
-Wdeprecated warning and expect a following note:.
From-SVN: r267941
Diffstat (limited to 'gcc/d')
-rw-r--r-- | gcc/d/ChangeLog | 5 | ||||
-rw-r--r-- | gcc/d/lang.opt | 2 |
2 files changed, 6 insertions, 1 deletions
diff --git a/gcc/d/ChangeLog b/gcc/d/ChangeLog index 6fa01f5..974b098 100644 --- a/gcc/d/ChangeLog +++ b/gcc/d/ChangeLog @@ -1,3 +1,8 @@ +2019-01-15 Richard Sandiford <richard.sandiford@arm.com> + + PR inline-asm/52813 + * lang.opt (Wdeprecated): Reference common.opt instead of c.opt. + 2019-01-12 Iain Buclaw <ibuclaw@gdcproject.org> * README.gcc: New file. diff --git a/gcc/d/lang.opt b/gcc/d/lang.opt index 52ddd77..83d3d21 100644 --- a/gcc/d/lang.opt +++ b/gcc/d/lang.opt @@ -124,7 +124,7 @@ Warn about casts that will produce a null result. Wdeprecated D -; Documented in C +; Documented in common.opt Werror D |