[llvm-branch-commits] [Support] parallelFor: enable caller participation (PR #209690)

[Fangrui Song via llvm-branch-commits]

MaskRay wrote:

> In principle, I think this is fine but it does not really feel right to continue to expose `getThreadIndex()`, so perhaps that should be removed if this change is accepted?

`getThreadIndex()` is no longer used and I can delete it in this patch, once 209687 is accepted.

> The only other thing that comes to mind is that the caller could get delayed by work initiated by another caller thread which could then introduce latency. But I do not believe that this situation arises right now.

Agreed.

> I haven't had much time to consider this in detail and will soon be on PTO for 2 weeks.

Thanks for the heads up!


https://github.com/llvm/llvm-project/pull/209690