https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89585
--- Comment #32 from Harald van Dijk <harald at gigawatt dot nl> --- (In reply to Jakub Jelinek from comment #30) > For 8.x we do not want to warn, that will break people building with -Werror. > For 9.x, Jason said that perhaps we should warn instead of error for > volatile, so I'll submit incremental patch. Good point about -Werror. The suggestion for a warning was based on it remaining an error in GCC 9. If it is going to be downgraded to a warning there, there's no point in a warning in GCC 8, so I'm happy with that approach. Thank you! One comment about the patch: it looks like resetting volatile_p (which you removed) and inline_p (which is still in) will have no effect, since the one use is conditional on parser->in_function_body anyway. Am I missing a use there? If not, is that in because the code is easier to understand if it is, or is it an oversight? The reset of goto_p affects how the remainder will be parsed, so that definitely does have an effect.
