DescriptionRevert of TaskGraphRunner Group support (patchset #12 id:360001 of https://codereview.chromium.org/1489233003/ )
Reason for revert:
Broke compile on Win x64 GN
https://build.chromium.org/p/chromium.win/builders/Win%20x64%20GN/builds/9223
Original issue's description:
> TaskGraphRunner Group support
>
> Adds a group to TaskGraphRunner, this can be used by an
> implementation along with priority in order to decide how to execute
> tasks.
>
> This is a prerequisite patch for thread affinity changes which are
> coming in a follow-up.
>
> This patch causes a moderate (~15%) regression in cc_perftests for TaskGraphRunner.
>
> The main regressions I'm seeing are:
>
> In ScheduleTasks, we're seeing an additional 3% time spent in operator[] for the two new maps vs ToT.
>
> We're also seeing an additional 5.5% of time spent in test code which builds the task graph.
>
> Other regressions are less clear, but probably have to do with the additional cost of ScheduleTasks in the function body itself (inlined looping).
>
> BUG=
> CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
>
> Committed: https://crrev.com/05ca4bc9df381e6345f9cf7ee29cdfa075be30c4
> Cr-Commit-Position: refs/heads/master@{#365140}
>
> Committed: https://crrev.com/3b4338fc32cf20703baa8066bdad7ce07c5ae696
> Cr-Commit-Position: refs/heads/master@{#365646}
TBR=haraken@chromium.org,oilpan-reviews@chromium.org,reveman@chromium.org,sievers@chromium.org,ericrk@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=
Committed: https://crrev.com/7e8a670ac2be0594334e7aad04384dd156add7e8
Cr-Commit-Position: refs/heads/master@{#365745}
Patch Set 1 #
Messages
Total messages: 6 (2 generated)
|