Qiskit / qiskit / 6568955708

87%

main: 88%

push

github

Pass manager refactoring: cleanup internals (#10127)

* Refactor internals of pass manager and flow controllers.

This PR introduces two changes for
* remove tight coupling of flow controller to pass runner instance,
* remove pass normalization.

PropertySet becomes context var so that flow controllers can be instantiated
without coupling to a pass runner instance. BasePass becomes an iterable
of length 1 to skip normalization. The decoupling of property set from
pass runner allows pass manager to directly broadcast pass runner
in the multithread, rather than distributing self and craete
multiple pass runner in each thread.

* Replace class attribute PASS_RUNNER with property method.

This allows subclass to dispatch different pass runner type depending on the target code.

* Refactor passmanager module

- Add OptimizerTask as a protocol for the pass and flow controller. These are in principle the same object that inputs and outputs IR with optimization.
- A task gains execute method that takes IR and property set. This makes property set local to iteration of the passes.
- Drop dependency on pass runner. Now pass manager has a single linear flow controller.
- Pass manager gain responsibility of compiler frontend and backend as pass runner dependency is removed. This allows flow controllers to be still type agnostic.
- Drop future property set, as this is no longer necessary because optimization task has the execute method explicitly taking the property set.

* Refactor transpiler passmanager

- Remove pass runner baseclass and replace RunningPassmanager baseclass with FlowControllerLiner
- Implemented frontoend and backend functionality in transpiler Pass manager

* Rename module: base_pass -> base_optimization_tasks

* Backward compatibility fix
- Move handling of required passes to generic pass itself. This makes optimizer tasks the composite pattern-like for more readability.
- Readd MetaPass to transpiler BasePass as a metaclass which implements ... (continued)

441 of 441 new or added lines in 10 files covered. (100.0%)

73789 of 84929 relevant lines covered (86.88%)

91837.84 hits per line