Update relevant CMakeList.txt.

PTAL.

Address comments.

Revert changes to Bazel file because it doesn't belong in this CL.

Address comments.

Move minTaskSize definition earlier to avoid contaminating the analysis results with the partially executed rewrite logic.

Same thing for another lambda.

Fix minor C++ compilation issue.

Address nit.

Elide dynamic check if possible.

Address comments: lift the scf.if out of the loops.

Rebase. Note that this made some of the loop unrolling logic dynamic which seeped all the way into loop building.

Change the comment.

In D115423#3181902, @mehdi_amini wrote:

The revision title mentions "AsyncParallelFor pass" but this isn't touching the pass itself right?

Improve docs.

Address comments.

Fix the failing test.

Rebase to head.

Use min/max ops.

Rebase.

Fix the test.

Appease clang.

Fix a typo.

Superflous update.

Address comments.

Addressing comments.

Address comments.

Use attributes as opposed to command line options.

In D106730#2907908, @ezhulenev wrote:

We'll need to rewrite public functions at some point as well (after resolving all the issues with TFRT threading). Alternatively instead of passing list of function names it can be a boolean flag allow-rewriting-public-functions.

Addressing comments.

Addressing comments.

Addressing comments.

Addressing comments.

Addressing comments.

Address clang warnings.

Aligned the calling convention of visit to match that of std::visit. Renamed the more convenient version of visit to match to have the best of the two worlds: better ergonomics and not clashing with std::visit. This time for real.

Include the VisitorTest.cpp into the CMakeList.txt.

Regarding deriving from using other kinds of functors: I have not yet seen a need for this. My preference is to cross that bridge when we reach it. :)

Aligned the calling convention of visit to match that of std::visit. Renamed the more convenient version of visit to match to have the best of the two worlds: better ergonomics and not clashing with std::visit.

Added docs with usage example. Hid implementation in detail namespace.

Replaced class with typename in template definition.

Replaced class with typename in template definition.

Re: handleAllErrors. Maybe. It might result in better runtime performance (because there is not recursion on the handlers), but it's also a rewrite. I am not sure if you are asking me to do something. :)

Thanks for taking a look, Mehdi and David! PTAL.

Added a unit test.

Added a unit test.