[WIP] Pipelines cleanup #30

PapyChacal · 2023-10-04T17:12:02Z

For now:

Simplify OpenMP pipeline (tested with MLIR, I only removed passes with absolutely no effect on the IR)
Use the same one for single thread, just remove the SCF->OpenMP pass. It had no impact on performance, but this removes an unnecessary dimension to performance investigations
Use Devito-inferred options to decide on using MPI/OpenMP. More importantly, this lift an arbitrary dependency on MPI4Py on all XDSLOperators.

…ffect once used right.)

More importantly, this lift an arbitrary dependency on mpi4py in unrelated cases.

review-notebook-app · 2023-10-04T17:14:58Z

Check out this pull request on

See visual diffs & provide feedback on Jupyter Notebooks.

Powered by ReviewNB

codecov · 2023-10-04T17:33:15Z

Codecov Report

All modified lines are covered by tests ✅

📢 Thoughts on this report? Let us know!.

georgebisbas · 2023-12-12T20:08:54Z

@PapyChacal is this needed?

PapyChacal · 2023-12-14T11:02:10Z

Not anymore, that is an old attempt at removing superfluous passes, AFAIU you redid it 🙂

PapyChacal added 3 commits October 4, 2023 17:22

Remove extra passes that had no effect on the IR.

2263129

Remove badly outdated comment (The pass did have important positive e…

36a635c

…ffect once used right.)

Merge CPU pipelines.

a9bacf9

PapyChacal self-assigned this Oct 4, 2023

PapyChacal changed the base branch from emilien/cleanup to master October 4, 2023 17:14

Use Devito options for OpenMP and MPI.

ac5dac3

More importantly, this lift an arbitrary dependency on mpi4py in unrelated cases.

PapyChacal force-pushed the emilien/pipelines branch from 6e0fc45 to ac5dac3 Compare October 4, 2023 17:14

PapyChacal changed the base branch from master to emilien/cleanup October 4, 2023 17:20

PapyChacal closed this Dec 14, 2023