ChangeLog

2012-01-20  Roland Takacs  <takacs.roland@stud.u-szeged.hu>

        [Qt] GC should be parallel on Qt platform

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

        Reviewed by NOBODY (OOPS!).

        These changes made the parallel gc feature available for Qt port.

        I inserted the "isMainThreadOrGCThread" function's symbol to the .order

        and .def files to build correctly on Windows platform.

        Furthermore I moved "registerGCThread" and "isMainThreadOrGCThread"

        implementations from the MainThreadMac.mm to MainThread.cpp to make

        parallel GC available not only on MAC platform.

        Measurement results:

        V8           speed-up:  1.071x as fast  [From: 746.1ms  To: 696.4ms ]

        WindScorpion speed-up:  1.082x as fast  [From: 3490.4ms To: 3226.7ms]

        V8 Splay     speed-up:  1.158x as fast  [From: 145.8ms  To: 125.9ms ]

        Tested on Intel(R) Core(TM) i5-2320 CPU @ 3.00GHz with 4-core.

        * JavaScriptCore.order:

        * JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.def:

        * wtf/MainThread.cpp:

        (WTF::registerGCThread):

        (WTF::isMainThreadOrGCThread):

        * wtf/MainThread.h:

        * wtf/Platform.h:

        * wtf/mac/MainThreadMac.mm:

2012-01-20  ChangSeok Oh  <shivamidow@gmail.com>

        undefined reference to symbol eina_module_free

Source/JavaScriptCore/JavaScriptCore.order

@@__ZN3WTF10StringImpl19createUninitializedEjRPt

__ZN3WTF5equalEPKNS_10StringImplES2_

__ZN3WTF10StringImpl6createEPKtj

__ZN3WTF12isMainThreadEv

43__ZN3WTF22isMainThreadOrGCThreadEv

__ZN3WTF10StringImplD1Ev

__ZN3WTF10StringImplD2Ev

__ZN3WTF6String6numberEy

Source/JavaScriptCore/JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.def

@@EXPORTS

    ?isDataDescriptor@PropertyDescriptor@JSC@@QBE_NXZ

    ?isHostFunctionNonInline@JSFunction@JSC@@ABE_NXZ

    ?isMainThread@WTF@@YA_NXZ

235 ?isMainThreadOrGCThread@WTF@@YA_NXZ

    ?isReachableFromOpaqueRoots@WeakHandleOwner@JSC@@UAE_NV?$Handle@W4Unknown@JSC@@@2@PAXAAVSlotVisitor@2@@Z

    ?isTerminatedExecutionException@JSC@@YA_NVJSValue@1@@Z

    ?isValidAllocation@Heap@JSC@@AAE_NI@Z

Source/JavaScriptCore/wtf/MainThread.cpp

@@bool isMainThread()

}

#endif

#if ENABLE(PARALLEL_GC)

static ThreadSpecific<bool>* isGCThread;

void registerGCThread()

{

    if (!isGCThread) {

        // This happens if we're running in a process that doesn't care about

        // MainThread.

        return;

    }

    **isGCThread = true;

}

bool isMainThreadOrGCThread()

{

    if (isGCThread->isSet() && **isGCThread)

        return true;

    return isMainThread();

}

#endif

252275} // namespace WTF

Source/JavaScriptCore/wtf/MainThread.h

@@WTF_EXPORT_PRIVATE bool isMainThread();

#if ENABLE(PARALLEL_GC)

void registerGCThread();

WTF_EXPORT_PRIVATE bool isMainThreadOrGCThread();

58 #elif PLATFORM(MAC)

59 bool isMainThreadOrGCThread();

#else

inline bool isMainThreadOrGCThread() { return isMainThread(); }

#endif

Source/JavaScriptCore/wtf/Platform.h

#define ENABLE_COMPARE_AND_SWAP 1

#endif

1121 #if !defined(ENABLE_PARALLEL_GC) && (PLATFORM(MAC) || PLATFORM(IOS)) && ENABLE(COMPARE_AND_SWAP)

1121#if !defined(ENABLE_PARALLEL_GC) && (PLATFORM(MAC) || PLATFORM(IOS) || PLATFORM(QT)) && ENABLE(COMPARE_AND_SWAP)

#define ENABLE_PARALLEL_GC 1

#endif

Source/JavaScriptCore/wtf/mac/MainThreadMac.mm

@@bool isMainThread()

    return pthread_equal(pthread_self(), mainThreadPthread);

}

#if ENABLE(PARALLEL_GC)

void registerGCThread()

{

    if (!isGCThread) {

        // This happens if we're running in a process that doesn't care about

        // MainThread.

        return;

    }

    **isGCThread = true;

}

bool isMainThreadOrGCThread()

{

    if (isGCThread->isSet() && **isGCThread)

        return true;

    return isMainThread();

}

#else

// This is necessary because JavaScriptCore.exp doesn't support preprocessor macros.

bool isMainThreadOrGCThread()

{

    return isMainThread();

}

#endif

175148} // namespace WTF