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

Harald van Dijk <harald at gigawatt dot nl> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #45910|0                           |1
        is obsolete|                            |

--- Comment #25 from Harald van Dijk <harald at gigawatt dot nl> ---
Created attachment 45912
  --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=45912&action=edit
gcc-8-pr89585-warn-v2.patch

Re-done based on the GCC 9 patch, modified to issue a warning for asm volatile
on GCC 8. Since Qt 5 has already been tested, I looked at Qt 4.8.7 and verified
that bitwise identical code is generated for what GCC 8.3 produces when the
volatile keyword is removed, and that the warning is issued when the volatile
keyword is left in:

../3rdparty/javascriptcore/JavaScriptCore/jit/JITStubs.cpp:483:5: warning: asm
qualifier outside of function body
 asm volatile (
     ^~~~~~~~
../3rdparty/javascriptcore/JavaScriptCore/jit/JITStubs.cpp:518:5: warning: asm
qualifier outside of function body
 asm volatile (
     ^~~~~~~~
../3rdparty/javascriptcore/JavaScriptCore/jit/JITStubs.cpp:534:5: warning: asm
qualifier outside of function body
 asm volatile (
     ^~~~~~~~

Reply via email to