[Webkit-unassigned] [Bug 73309] [Qt] GC should be parallel on Qt platform

[bugzilla-daemon at webkit.org]

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


Zoltan Herczeg <zherczeg at webkit.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #122739|review?                     |review-
               Flag|                            |




--- Comment #42 from Zoltan Herczeg <zherczeg at webkit.org>  2012-01-18 00:53:27 PST ---
(From update of attachment 122739)
View in context: https://bugs.webkit.org/attachment.cgi?id=122739&action=review

> Source/JavaScriptCore/wtf/MainThread.cpp:-92
> -
>  #if !PLATFORM(MAC)
> -

Why did you removed these lines?

> Source/JavaScriptCore/wtf/MainThread.cpp:-107
> -
>  #else
> -

Ditto.

> Source/JavaScriptCore/wtf/MainThread.cpp:255
> +        // This happens if we're running in a process that doesn't care about
> +        // MainThread.

Could you write a better explanation here?

Ok, basically I would split this patch into two:

One which adds the number of processor cores part, and a followup patch which adds the parallel gc feature.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.