Do not unroll gates in basis in add_control (backport #13475)

[mergify]

Summary

The add_control mechanism currently does some redundant unrolling and re-wrapping of gates that are already in the basis of gates we know how to control. This has knock-on effects and causes #13473, by translating an RZ operation (already supported) to a Phase gate.

This PR avoids this unrolling. As discussed with @alexanderivrii, in the longer term we should consider rewriting this logic entirely and have a controlled gate plugin, which also allows users to supply their own control mechanism.

Details and comments

• There's a failing test (likely since the decomposition now uses the RX-path instead of wrongly the RX->R->U3->U path before), but the unitary is still correct and there are only dangling U(0,0,0) gates.
• Need to add a test for The multi-controlled gate by RZGate().control() is less optimized. #13473

---

This is an automatic backport of pull request #13475 done by [Mergify](https://mergify.com).

[qiskit-bot]

Thank you for opening a new pull request.

Before your PR can be merged it will first need to pass continuous integration tests and be reviewed. Sometimes the review process can be slow, so please be patient.

While you're waiting, please feel free to review other open PRs. While only a subset of people are authorized to approve pull requests for merging, everyone is encouraged to review open pull requests. Doing reviews helps reduce the burden on the core team and helps make the project's code better for everyone.

One or more of the following people are relevant to this code:

• @Qiskit/terra-core