2008-08-10  Jan Michael Alonzo  <jmalonzo@webkit.org>

        Reviewed (and updated) by Alp Toker.

        https://bugs.webkit.org/show_bug.cgi?id=16620

        [GTK] Autotools make dist and make check support

        Get make dist working.

        Note that not all possible configurations have been tested yet.

        * GNUmakefile.am:

2008-08-09  Alexey Proskuryakov  <ap@webkit.org>

        Reviewed by Sam Weinig.

        Added same heap debug checks to more code paths.

        * kjs/JSActivation.cpp:

        (KJS::JSActivation::put):

        (KJS::JSActivation::putWithAttributes):

        * kjs/JSGlobalObject.cpp:

        (KJS::JSGlobalObject::putWithAttributes):

        * kjs/JSObject.h:

        (KJS::JSObject::putDirect):

        * kjs/JSVariableObject.h:

        (KJS::JSVariableObject::symbolTablePut):

        (KJS::JSVariableObject::symbolTablePutWithAttributes):

2008-08-09  Cameron Zwarich  <cwzwarich@uwaterloo.ca>

        Reviewed by Maciej.

        Fix some style issues in the sampling tool.

        * VM/SamplingTool.cpp:

        (KJS::sleepForMicroseconds):

        (KJS::SamplingTool::dump):

2008-08-09  Cameron Zwarich  <cwzwarich@uwaterloo.ca>

        Reviewed by Oliver.

        Revision 35651, despite being a rather trivial change, introduced a

        large regression on the regexp-dna SunSpider test. This regression

        stemmed from an increase in the size of CodeBlock::dump(). There is

        no reason for this method (and several related methods) to be compiled

        in non-debug builds with the sampling tool disabled. This patch

        conditionally compiles them, reversing the regression on SunSpider.

        * JavaScriptCore.exp:

        * VM/CodeBlock.cpp:

        * VM/CodeBlock.h:

        * VM/Machine.cpp:

2008-08-08  Cameron Zwarich  <cwzwarich@uwaterloo.ca>

        Reviewed by Oliver.

        Bug 20330: JSCore crash loading any filehurricane media page

        <https://bugs.webkit.org/show_bug.cgi?id=20330>

        Fix a typo in the constant loading patch. Also, add a case for

        op_unexpected_load to CodeBlock::dump().

        * VM/CodeBlock.cpp:

        (KJS::CodeBlock::dump):

        * VM/CodeGenerator.cpp:

        (KJS::CodeGenerator::addUnexpectedConstant):

2008-08-08  Matt Lilek  <webkit@mattlilek.com>

        Not reviewed, build fix.

        * JavaScriptCore.exp:

2008-08-08  Oliver Hunt  <oliver@apple.com>

        Reviewed by Cameron Zwarich.

        Improve performance of arithmetic operators

        Added a fast (non-virtual) mechanism to determine if a non-immediate JSValue*

        is a JSNumberCell.  We then use this to allow improved specialisation in many

        arithmetic operators.  SunSpider reports a 2.5% progression overall, with greater

        than 10% progressions on a number of arithmetic heavy tests.

        * VM/Machine.cpp:

        (KJS::fastIsNumber):

        (KJS::fastToInt32):

        (KJS::fastToUInt32):

        (KJS::jsLess):

        (KJS::jsLessEq):

        (KJS::jsAdd):

        (KJS::Machine::privateExecute):

        * kjs/JSNumberCell.h:

        (KJS::JSNumberCell::fastToInt32):

        (KJS::JSNumberCell::fastToUInt32):

        * kjs/collector.cpp:

        (KJS::allocateBlock):

        (KJS::Heap::heapAllocate):

        * kjs/collector.h:

        (KJS::Heap::fastIsNumber):

2008-08-06  Adam Roben  <aroben@apple.com>

        Try to fix the Windows build bots

        * API/JSBase.cpp: Touch this to force JSC to rebuild and re-copy the

        WTF headers.

2008-08-06  Tor Arne Vestbø  <tavestbo@trolltech.com>

        Revert change 35595.

        * wtf/RetainPtr.h:

2008-08-06  Ariya Hidayat  <ariya.hidayat@trolltech.com>

        Fix non-Mac build.

        * wtf/RetainPtr.h: CoreFoundation only for PLATFORM(MAC)

2008-08-06  Ariya Hidayat  <ariya.hidayat@trolltech.com>

        Fix non-Mac build.

        * wtf/RetainPtr.h: CoreFoundation only for PLATFORM(MAC)

2008-08-06  Csaba Osztrogonac  <oszi@inf.u-szeged.hu>

        Reviewed by Darin. Landed by Cameron.

        Bug 20272: typo in JavaScriptCore

        <https://bugs.webkit.org/show_bug.cgi?id=20272>

        Correct the documentation for op_not. (typo)

        Fix #undef. (typo)

        * VM/Machine.cpp:

        (KJS::Machine::privateExecute):

2008-08-06  Cameron Zwarich  <cwzwarich@webkit.org>

        Reviewed by Maciej.

        Bug 20286: Load constants all at once instead of using op_load

        <https://bugs.webkit.org/show_bug.cgi?id=20286>

        Load constants all at once into temporary registers instead of using

        individual instances of op_load.

        This is a 2.6% speedup on SunSpider.

        * JavaScriptCore.exp:

        * VM/CodeBlock.cpp:

        (KJS::CodeBlock::dump):

        (KJS::CodeBlock::mark):

        * VM/CodeBlock.h:

        * VM/CodeGenerator.cpp:

        (KJS::CodeGenerator::CodeGenerator):

        (KJS::CodeGenerator::newTemporary):

        (KJS::CodeGenerator::addConstant):

        (KJS::CodeGenerator::addUnexpectedConstant):

        (KJS::CodeGenerator::emitLoad):

        (KJS::CodeGenerator::emitUnexpectedLoad):

        (KJS::CodeGenerator::emitNewError):

        * VM/CodeGenerator.h:

        * VM/Machine.cpp:

        (KJS::slideRegisterWindowForCall):

        (KJS::Machine::unwindCallFrame):

        (KJS::Machine::throwException):

        (KJS::Machine::execute):

        (KJS::Machine::privateExecute):

        * VM/Machine.h:

        * VM/Opcode.h:

        * VM/RegisterID.h:

        (KJS::RegisterID::RegisterID):

        (KJS::RegisterID::makeConstant):

        (KJS::RegisterID::isTemporary):

        * kjs/NodeInfo.h:

        * kjs/Parser.cpp:

        (KJS::Parser::didFinishParsing):

        * kjs/Parser.h:

        (KJS::Parser::parse):

        * kjs/grammar.y:

        * kjs/nodes.cpp:

        (KJS::NullNode::emitCode):

        (KJS::BooleanNode::emitCode):

        (KJS::NumberNode::emitCode):

        (KJS::StringNode::emitCode):

        (KJS::ArrayNode::emitCode):

        (KJS::DeleteResolveNode::emitCode):

        (KJS::DeleteValueNode::emitCode):

        (KJS::VoidNode::emitCode):

        (KJS::ConstDeclNode::emitCodeSingle):

        (KJS::ReturnNode::emitCode):

        (KJS::ScopeNode::ScopeNode):

        (KJS::ProgramNode::ProgramNode):

        (KJS::ProgramNode::create):

        (KJS::EvalNode::EvalNode):

        (KJS::EvalNode::create):

        (KJS::FunctionBodyNode::FunctionBodyNode):

        (KJS::FunctionBodyNode::create):

        (KJS::FunctionBodyNode::emitCode):

        * kjs/nodes.h:

        (KJS::ScopeNode::neededConstants):

2008-08-05  Maciej Stachowiak  <mjs@apple.com>

        Reviewed by Cameron.

        - add fast path for immediates to % operator, as we have for many other math ops

        This fixes handling for a 0 divisor relative to the last patch. Only an 0.2% speedup on SunSpider but

        still a 1.4x win on Oliver's prime test.

        * VM/Machine.cpp:

        (KJS::Machine::privateExecute):

2008-08-05  Cameron Zwarich  <cwzwarich@uwaterloo.ca>

        Reviewed by Darin.

        Bug 20293: Crash in JavaScript codegen for eval("const a;")

        <https://bugs.webkit.org/show_bug.cgi?id=20293>

        Correctly handle constant declarations in eval code with no initializer.

        * kjs/nodes.cpp:

        (KJS::ConstDeclNode::emitCodeSingle):

2008-08-05  Cameron Zwarich  <cwzwarich@uwaterloo.ca>

        Reviewed by Oliver.

        Roll out r35555 because of correctness issues.

        * VM/Machine.cpp:

        (KJS::Machine::privateExecute):

2008-08-05  Maciej Stachowiak  <mjs@apple.com>

        Reviewed by Geoff.

        - add fast path for immediates to % operator, as we have for many other math ops

        0.6% speedup on SunSpider. 1.4x speedup on a prime testing torture test that Oliver whipped up.

        * VM/Machine.cpp:

        (KJS::Machine::privateExecute):

2008-07-31  Oliver Hunt  <oliver@apple.com>

        Reviewed by Cameron Zwarich.

        Bug 19359: JavaScriptCore behaves differently from FF2/3 and IE when handling context in catch statement

        <https://bugs.webkit.org/show_bug.cgi?id=19359>

        Make our catch behave like Firefox and IE, we do this by using a StaticScopeObject

        instead of a generic JSObject for the scope node.  We still don't make use of the

        fact that we have a static scope inside the catch block, so the internal performance

        of the catch block is not improved, even though technically it would be possible to

        do so.

        * VM/CodeBlock.cpp:

        (KJS::CodeBlock::dump):

        * VM/CodeGenerator.cpp:

        (KJS::CodeGenerator::emitPushNewScope):

        * VM/CodeGenerator.h:

        * VM/Machine.cpp:

        (KJS::createExceptionScope):

        (KJS::Machine::privateExecute):

        * VM/Machine.h:

        * VM/Opcode.h:

        * kjs/JSStaticScopeObject.cpp:

        (KJS::JSStaticScopeObject::toThisObject):

        (KJS::JSStaticScopeObject::put):

        * kjs/JSStaticScopeObject.h:

        * kjs/nodes.cpp:

        (KJS::TryNode::emitCode):

2008-08-02  Rob Gowin  <robg@gowin.net>

        Reviewed by Eric Seidel.

        Added JavaScriptCore/API/WebKitAvailability to list of files in

        javascriptcore_h_api.

        * GNUmakefile.am:

2008-08-01  Alexey Proskuryakov  <ap@webkit.org>

        Rubber-stamped by Maciej.

        Remove JSGlobalData::DataInstance. It was only needed when we had per-thread JSGlobalData

        instances.

        * kjs/JSGlobalData.h:

2008-07-31  Kevin Ollivier <kevino@theolliviers.com>

        Second attempt at Windows/wx build fix. Instead of avoiding inclusion of windows.h,

        use defines, etc. to avoid conflicts in each affected file. Also, change PLATFORM(WIN)

        to PLATFORM(WIN_OS) so that other ports using Windows headers get the right impls.

        * VM/SamplingTool.cpp:

        * wtf/Threading.h:

2008-07-31  Anders Carlsson  <andersca@apple.com>

        Reviewed by Adam.

        Fix Windows build.

        * kjs/collector.h:

        * wtf/FastMalloc.cpp:

2008-07-31  Csaba Osztrogonac  <oszi@inf.u-szeged.hu>

        Reviewed by Simon.

        Bug 20170: [Qt] missing namespace defines in JavaScriptCore.pro

        <https://bugs.webkit.org/show_bug.cgi?id=20170>

        * JavaScriptCore.pro: Added missing define.

2008-07-31  Alexey Proskuryakov  <ap@webkit.org>

        Rubber-stamped by Maciej.

        Eliminate JSLock (it was already disabled, removing the stub implementaion and all

        call sites now).

        * API/JSBase.cpp:

        (JSEvaluateScript):

        (JSCheckScriptSyntax):

        (JSGarbageCollect):

        * API/JSCallbackConstructor.cpp:

        (KJS::constructJSCallback):

        * API/JSCallbackFunction.cpp:

        (KJS::JSCallbackFunction::call):

        * API/JSCallbackObjectFunctions.h:

        (KJS::::init):

        (KJS::::getOwnPropertySlot):

        (KJS::::put):

        (KJS::::deleteProperty):

        (KJS::::construct):

        (KJS::::hasInstance):

        (KJS::::call):

        (KJS::::getPropertyNames):

        (KJS::::toNumber):

        (KJS::::toString):

        (KJS::::staticValueGetter):

        (KJS::::callbackGetter):

        * API/JSContextRef.cpp:

        (JSGlobalContextCreateInGroup):

        (JSGlobalContextRetain):

        (JSGlobalContextRelease):

        * API/JSObjectRef.cpp:

        (JSObjectMake):

        (JSObjectMakeFunctionWithCallback):

        (JSObjectMakeConstructor):

        (JSObjectMakeFunction):

        (JSObjectHasProperty):

        (JSObjectGetProperty):

        (JSObjectSetProperty):

        (JSObjectGetPropertyAtIndex):

        (JSObjectSetPropertyAtIndex):

        (JSObjectDeleteProperty):

        (JSObjectCallAsFunction):

        (JSObjectCallAsConstructor):

        (JSObjectCopyPropertyNames):

        (JSPropertyNameArrayRelease):

        (JSPropertyNameAccumulatorAddName):

        * API/JSStringRef.cpp:

        (JSStringRelease):

        * API/JSValueRef.cpp:

        (JSValueIsEqual):

        (JSValueIsInstanceOfConstructor):

        (JSValueMakeNumber):

        (JSValueMakeString):

        (JSValueToNumber):

        (JSValueToStringCopy):

        (JSValueToObject):

        (JSValueProtect):

        (JSValueUnprotect):

        * ForwardingHeaders/JavaScriptCore/JSLock.h: Removed.

        * GNUmakefile.am:

        * JavaScriptCore.exp:

        * JavaScriptCore.order:

        * JavaScriptCore.pri:

        * JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.vcproj:

        * JavaScriptCore.xcodeproj/project.pbxproj:

        * JavaScriptCoreSources.bkl:

        * kjs/AllInOneFile.cpp:

        * kjs/JSGlobalData.cpp:

        (KJS::JSGlobalData::JSGlobalData):

        * kjs/JSGlobalData.h:

        * kjs/JSGlobalObject.cpp:

        (KJS::JSGlobalObject::~JSGlobalObject):

        (KJS::JSGlobalObject::init):

        * kjs/JSLock.cpp: Removed.

        * kjs/JSLock.h: Removed.

        * kjs/Shell.cpp:

        (functionGC):

        (jscmain):

        * kjs/collector.cpp:

        (KJS::Heap::~Heap):

        (KJS::Heap::heapAllocate):

        (KJS::Heap::setGCProtectNeedsLocking):

        (KJS::Heap::protect):

        (KJS::Heap::unprotect):

        (KJS::Heap::collect):

        * kjs/identifier.cpp:

        * kjs/interpreter.cpp:

        (KJS::Interpreter::checkSyntax):

        (KJS::Interpreter::evaluate):

2008-07-31  Alexey Proskuryakov  <ap@webkit.org>

        Rubber-stamped by Oliver Hunt.

        Fix the Mac project to not display "test/" as part of file name for tests.

        * JavaScriptCore.xcodeproj/project.pbxproj:

2008-07-31  Eric Seidel  <eric@webkit.org>

        Reviewed by Alexey Proskuryakov.

        Rename USE(MULTIPLE_THREADS) to ENABLE(JSC_MULTIPLE_THREADS)

        to better match the use/enable pattern (and better describe

        the usage of the feature in question.)

        I also fixed a couple other ENABLE_ macros to be pre-processor

        definition override-able to match the rest of the ENABLE_ macros

        since it seems to be our convention that build systems can set

        ENABLE_ macros in Makefiles.

        * kjs/InitializeThreading.cpp:

        (KJS::initializeThreadingOnce):

        * kjs/JSGlobalData.cpp:

        (KJS::JSGlobalData::JSGlobalData):

        (KJS::JSGlobalData::~JSGlobalData):

        * kjs/MathObject.cpp:

        * kjs/collector.cpp:

        (KJS::Heap::Heap):

        (KJS::Heap::~Heap):

        (KJS::allocateBlock):

        (KJS::Heap::markStackObjectsConservatively):

        * kjs/collector.h:

        * kjs/dtoa.cpp:

        (KJS::pow5mult):

        (KJS::rv_alloc):

        (KJS::freedtoa):

        (KJS::dtoa):

        * wtf/FastMalloc.cpp:

        * wtf/Platform.h:

        * wtf/RefCountedLeakCounter.cpp:

2008-07-30  Eric Seidel  <eric@webkit.org>

        Reviewed by Mark Rowe.

        Try to clean up our usage of USE(MULTIPLE_THREADS) vs. USE(PTHREADS) a little.

        It looks like JSC assumes that if MULTIPLE_THREADS is defined, then pthreads will always be available

        I'm not sure that's always the case for gtk, certainly not for Windows.  We should eventually go back

        and fix wtf/Threading.h to cover all these cases some day.

        * kjs/JSLock.cpp:

        * kjs/collector.h:

        * wtf/Platform.h:

2008-07-30  Eric Seidel  <eric@webkit.org>

        Reviewed by Oliver.

        MSVC warns when structs are called classes or vice versa.

        Make all the source refer to JSGlobalData as a class.

        * kjs/CommonIdentifiers.h:

        * kjs/JSGlobalData.h:

        * kjs/Parser.h:

        * kjs/lexer.h:

2008-07-30  Alexey Proskuryakov  <ap@webkit.org>

        Reviewed by Geoff Garen.

        Add consistency checks to UString to document and enforce its design.

        * kjs/ustring.cpp:

        (KJS::UString::Rep::create):

        (KJS::UString::Rep::destroy):

        (KJS::UString::Rep::checkConsistency):

        (KJS::UString::expandCapacity):

        (KJS::UString::expandPreCapacity):

        (KJS::UString::UString):

        (KJS::UString::spliceSubstringsWithSeparators):

        (KJS::UString::append):

        * kjs/ustring.h:

        (KJS::UString::Rep::checkConsistency):

2008-07-30  Gavin Barraclough  <barraclough@apple.com>

        Reviewed by Geoff Garen.

        Fixes for Windows and non-AllInOne file build with SamplingTool, plus review fixes.

        * GNUmakefile.am:              Adding SamplingTool.cpp to build.

        * JavaScriptCore.exp:          Export hooks to init & control SamplingTool.

        * JavaScriptCore.pri:          Adding SamplingTool.cpp to build.

        * JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.vcproj: Adding SamplingTool.cpp to build.

        * JavaScriptCore.xcodeproj/project.pbxproj: Adding SamplingTool.cpp to build.

        * JavaScriptCoreSources.bkl:   Adding SamplingTool.cpp to build.

        * VM/Machine.cpp:              MACHINE_SAMPLING_callingNativeFunction renamed MACHINE_SAMPLING_callingHostFunction

        * VM/Machine.h:

        * VM/Opcode.cpp:               SamplingTool moved to SamplingTool.cpp/.h, opcodeNames generated from FOR_EACH_OPCODE_ID.

        * VM/Opcode.h:

        * VM/SamplingTool.cpp:         Added .cpp/.h for SamplingTool.

        * VM/SamplingTool.h:

        * kjs/Shell.cpp:               Switched SAMPLING_TOOL_ENABLED to ENABLE_SAMPLING_TOOL.

        * wtf/Platform.h:              Added ENABLE_SAMPLING_TOOL config option.

        * kjs/nodes.cpp:               Header include to fix non-AllInOne builds.

2008-07-30  Ariya Hidayat  <ariya.hidayat@trolltech.com>