[PATCH] [lld][Core] Implement parallel_for_each

Davide Italiano davide at freebsd.org
Mon Mar 16 12:07:06 PDT 2015


On Mon, Mar 16, 2015 at 12:04 PM, Davide Italiano <davide at freebsd.org> wrote:
> On Mon, Mar 16, 2015 at 11:50 AM, Shankar Easwaran
> <shankare at codeaurora.org> wrote:
>> Agree, we need to debug this, but I am unaware of tools that can show how
>> threads are being scheduled ? Any pointers ?
>>
>
> As I mentioned in the LLD performance thread I'm aware of Vtune that
> does that (if you have a license for it). It has also a very nice
> 'lock analysis' feature that might help in this case.
> That said, I vote for getting the very first version you proposed in
> the tree and refine things later, unless there's strong opposition. No
> particular benefits of regression were found, but the patch I built on
> top of it (linked in the other thread) showed up a significant
> benchmarks for my use case.
>

This should be read as "showed a significant speedup", sorry.
Also, I'm definitely in favour of having an iterative rather than
recursive version (if others agree and you can build an iterative
version of the very first patch I will try getting some #s later
today).

-- 
Davide

"There are no solved problems; there are only problems that are more
or less solved" -- Henri Poincare



More information about the llvm-commits mailing list