Title: [180719] trunk/Source/_javascript_Core
Revision
180719
Author
[email protected]
Date
2015-02-26 18:07:02 -0800 (Thu, 26 Feb 2015)

Log Message

Assertion fix for r180711: The bool returning form of BytecodeGenerator::addVar() can be removed.
<https://webkit.org/b/142064>

Reviewed by Joseph Pecoraro.

* bytecompiler/BytecodeGenerator.cpp:
(JSC::BytecodeGenerator::addVar):

Modified Paths

Diff

Modified: trunk/Source/_javascript_Core/ChangeLog (180718 => 180719)


--- trunk/Source/_javascript_Core/ChangeLog	2015-02-27 01:57:48 UTC (rev 180718)
+++ trunk/Source/_javascript_Core/ChangeLog	2015-02-27 02:07:02 UTC (rev 180719)
@@ -1,5 +1,15 @@
 2015-02-26  Mark Lam  <[email protected]>
 
+        Assertion fix for r180711: The bool returning form of BytecodeGenerator::addVar() can be removed.
+        <https://webkit.org/b/142064>
+
+        Reviewed by Joseph Pecoraro.
+
+        * bytecompiler/BytecodeGenerator.cpp:
+        (JSC::BytecodeGenerator::addVar):
+
+2015-02-26  Mark Lam  <[email protected]>
+
         MachineThreads::Thread clean up has a use after free race condition.
         <https://webkit.org/b/141990>
 

Modified: trunk/Source/_javascript_Core/bytecompiler/BytecodeGenerator.cpp (180718 => 180719)


--- trunk/Source/_javascript_Core/bytecompiler/BytecodeGenerator.cpp	2015-02-27 01:57:48 UTC (rev 180718)
+++ trunk/Source/_javascript_Core/bytecompiler/BytecodeGenerator.cpp	2015-02-27 02:07:02 UTC (rev 180719)
@@ -142,9 +142,10 @@
         m_watchableVariables.append(ident);
     }
     
+    RegisterID* regID = addVar();
     ASSERT(watchMode == NotWatchable || static_cast<size_t>(m_codeBlock->m_numVars) == m_watchableVariables.size());
     
-    return addVar();
+    return regID;
 }
 
 BytecodeGenerator::BytecodeGenerator(VM& vm, ProgramNode* programNode, UnlinkedProgramCodeBlock* codeBlock, DebuggerMode debuggerMode, ProfilerMode profilerMode)
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to