Title: [109511] trunk/Source/_javascript_Core
- Revision
- 109511
- Author
- [email protected]
- Date
- 2012-03-01 22:28:45 -0800 (Thu, 01 Mar 2012)
Log Message
The JIT should not crash the entire process just because there is not enough executable
memory, if the LLInt is enabled
https://bugs.webkit.org/show_bug.cgi?id=79962
Patch by Filip Pizlo <[email protected]> on 2012-03-01
Reviewed by Csaba Osztrogonác.
Fix for ARM, SH4.
* assembler/AssemblerBufferWithConstantPool.h:
(JSC::AssemblerBufferWithConstantPool::executableCopy):
Modified Paths
Diff
Modified: trunk/Source/_javascript_Core/ChangeLog (109510 => 109511)
--- trunk/Source/_javascript_Core/ChangeLog 2012-03-02 06:28:17 UTC (rev 109510)
+++ trunk/Source/_javascript_Core/ChangeLog 2012-03-02 06:28:45 UTC (rev 109511)
@@ -1,3 +1,16 @@
+2012-03-01 Filip Pizlo <[email protected]>
+
+ The JIT should not crash the entire process just because there is not enough executable
+ memory, if the LLInt is enabled
+ https://bugs.webkit.org/show_bug.cgi?id=79962
+
+ Reviewed by Csaba Osztrogonác.
+
+ Fix for ARM, SH4.
+
+ * assembler/AssemblerBufferWithConstantPool.h:
+ (JSC::AssemblerBufferWithConstantPool::executableCopy):
+
2012-03-01 Ryosuke Niwa <[email protected]>
Revert my change. Broke builds.
Modified: trunk/Source/_javascript_Core/assembler/AssemblerBufferWithConstantPool.h (109510 => 109511)
--- trunk/Source/_javascript_Core/assembler/AssemblerBufferWithConstantPool.h 2012-03-02 06:28:17 UTC (rev 109510)
+++ trunk/Source/_javascript_Core/assembler/AssemblerBufferWithConstantPool.h 2012-03-02 06:28:45 UTC (rev 109511)
@@ -195,10 +195,10 @@
putIntegralUnchecked(value.low);
}
- PassRefPtr<ExecutableMemoryHandle> executableCopy(JSGlobalData& globalData, void* ownerUID)
+ PassRefPtr<ExecutableMemoryHandle> executableCopy(JSGlobalData& globalData, void* ownerUID, JITCompilationEffort effort)
{
flushConstantPool(false);
- return AssemblerBuffer::executableCopy(globalData, ownerUID);
+ return AssemblerBuffer::executableCopy(globalData, ownerUID, effort);
}
void putShortWithConstantInt(uint16_t insn, uint32_t constant, bool isReusable = false)
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes
