Changeset 112460 in webkit for trunk/Source/JavaScriptCore

Timestamp:

Mar 28, 2012, 3:43:04 PM (13 years ago)

Author:

[email protected]

Message:

Build fix - some compiles generating NORETURN related warnings.

• yarr/YarrJIT.cpp:

(JSC::Yarr::YarrGenerator::setSubpatternStart):
(JSC::Yarr::YarrGenerator::setSubpatternEnd):
(JSC::Yarr::YarrGenerator::clearSubpatternStart):

Location:

trunk/Source/JavaScriptCore

Files:

• ChangeLog (modified) (1 diff)
• yarr/YarrJIT.cpp (modified) (3 diffs)

trunk/Source/JavaScriptCore/ChangeLog

@@ +1 @@
+2012-03-28  Gavin Barraclough  <[email protected]>
+
+        Build fix - some compiles generating NORETURN related warnings.
+
+        * yarr/YarrJIT.cpp:
+        (JSC::Yarr::YarrGenerator::setSubpatternStart):
+        (JSC::Yarr::YarrGenerator::setSubpatternEnd):
+        (JSC::Yarr::YarrGenerator::clearSubpatternStart):
+
 2012-03-28  Kevin Ollivier  <[email protected]>
 

trunk/Source/JavaScriptCore/yarr/YarrJIT.cpp

@@ -329 +329 @@
     {
         ASSERT(subpattern);
-        ASSERT(compileMode == IncludeSubpatterns);
+        // FIXME: should be able to ASSERT(compileMode == IncludeSubpatterns), but then this function is conditionally NORETURN. :-(
         store32(reg, Address(output, (subpattern << 1) * sizeof(int)));
     }
@@ -335 +335 @@
     {
         ASSERT(subpattern);
-        ASSERT(compileMode == IncludeSubpatterns);
+        // FIXME: should be able to ASSERT(compileMode == IncludeSubpatterns), but then this function is conditionally NORETURN. :-(
         store32(reg, Address(output, ((subpattern << 1) + 1) * sizeof(int)));
     }
@@ -341 +341 @@
     {
         ASSERT(subpattern);
-        ASSERT(compileMode == IncludeSubpatterns);
+        // FIXME: should be able to ASSERT(compileMode == IncludeSubpatterns), but then this function is conditionally NORETURN. :-(
         store32(TrustedImm32(-1), Address(output, (subpattern << 1) * sizeof(int)));
     }