https://gcc.gnu.org/bugzilla/show_bug.cgi?id=52813

--- Comment #10 from Eric Gallager <egallager at gcc dot gnu.org> ---
(In reply to rsand...@gcc.gnu.org from comment #8)
> Author: rsandifo
> Date: Tue Jan 15 16:46:54 2019
> New Revision: 267941
> 
> URL: https://gcc.gnu.org/viewcvs?rev=267941&root=gcc&view=rev
> Log:
> 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.sandif...@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:.
> 
> Modified:
>     trunk/gcc/ChangeLog
>     trunk/gcc/c-family/ChangeLog
>     trunk/gcc/c-family/c.opt
>     trunk/gcc/cfgexpand.c
>     trunk/gcc/common.opt
>     trunk/gcc/d/ChangeLog
>     trunk/gcc/d/lang.opt
>     trunk/gcc/doc/extend.texi
>     trunk/gcc/testsuite/ChangeLog
>     trunk/gcc/testsuite/gcc.target/i386/pr52813.c

(In reply to Christophe Lyon from comment #9)
> Author: clyon
> Date: Fri Jan 18 09:57:41 2019
> New Revision: 268066
> 
> URL: https://gcc.gnu.org/viewcvs?rev=268066&root=gcc&view=rev
> Log:
> [ARM][testsuite] follow-up to PR target/52813 and target/11807 fix.
> 
> 2019-01-18  Christophe Lyon  <christophe.l...@linaro.org>
> 
>       * gcc.target/arm/pr77904.c: Add dg-warning for sp clobber.
> 
> 
> Modified:
>     trunk/gcc/testsuite/ChangeLog
>     trunk/gcc/testsuite/gcc.target/arm/pr77904.c

So after these 2, has this been fixed now?

Reply via email to