On 3/7/19 2:13 PM, Jakub Jelinek wrote:
On Wed, Mar 06, 2019 at 07:44:25PM -0500, Jason Merrill wrote:
In addition to that, it mentions in the documentation that qualifiers are
not allowed at toplevel asm statements; apparently our documentation at
least from r220506 for GCC 5 says that at toplevel Basic Asm needs to be
used and for Basic Asm lists volatile qualifier as optional and its behavior
(that it is ignored for Basic Asm).  Makes me wonder if we don't want to
keep accepting/ignoring volatile at toplevel for both C and C++ instead of
rejecting it (and rejecting just the other qualifiers).  Thoughts on this?

That seems reasonable.  Or using warning or permerror instead of error.

This incremental patch uses warning.  Bootstrapped/regtested on
x86_64-linux and i686-linux, ok for trunk?

OK.

Jason

Reply via email to