Title: [236813] trunk/Source/_javascript_Core
Revision
236813
Author
[email protected]
Date
2018-10-03 15:02:44 -0700 (Wed, 03 Oct 2018)

Log Message

Suppress unreachable code warning for LLIntAssembly.h code.
https://bugs.webkit.org/show_bug.cgi?id=190263
<rdar://problem/44986532>

Reviewed by Saam Barati.

This is needed because LLIntAssembly.h is template generated from LowLevelInterpreter
asm files, and may contain dead code which are harmless, but will trip up the warning.
We should suppress the warning so that it doesn't break builds.

* llint/LowLevelInterpreter.cpp:
(JSC::CLoop::execute):

Modified Paths

Diff

Modified: trunk/Source/_javascript_Core/ChangeLog (236812 => 236813)


--- trunk/Source/_javascript_Core/ChangeLog	2018-10-03 21:09:42 UTC (rev 236812)
+++ trunk/Source/_javascript_Core/ChangeLog	2018-10-03 22:02:44 UTC (rev 236813)
@@ -1,3 +1,18 @@
+2018-10-03  Mark Lam  <[email protected]>
+
+        Suppress unreachable code warning for LLIntAssembly.h code.
+        https://bugs.webkit.org/show_bug.cgi?id=190263
+        <rdar://problem/44986532>
+
+        Reviewed by Saam Barati.
+
+        This is needed because LLIntAssembly.h is template generated from LowLevelInterpreter
+        asm files, and may contain dead code which are harmless, but will trip up the warning.
+        We should suppress the warning so that it doesn't break builds.
+
+        * llint/LowLevelInterpreter.cpp:
+        (JSC::CLoop::execute):
+
 2018-10-03  Dan Bernstein  <[email protected]>
 
         _javascript_Core part of [Xcode] Update some build settings as recommended by Xcode 10

Modified: trunk/Source/_javascript_Core/llint/LowLevelInterpreter.cpp (236812 => 236813)


--- trunk/Source/_javascript_Core/llint/LowLevelInterpreter.cpp	2018-10-03 21:09:42 UTC (rev 236812)
+++ trunk/Source/_javascript_Core/llint/LowLevelInterpreter.cpp	2018-10-03 22:02:44 UTC (rev 236813)
@@ -446,7 +446,9 @@
         // bytecode handlers for the interpreter, as compiled from
         // LowLevelInterpreter.asm and its peers.
 
+        IGNORE_CLANG_WARNINGS_BEGIN("unreachable-code")
         #include "LLIntAssembly.h"
+        IGNORE_CLANG_WARNINGS_END
 
         OFFLINE_ASM_GLUE_LABEL(llint_return_to_host)
         {
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to