Consider using transient storage in Governor and AccessManager

[ernestognw]

📝 Details

Similar to #4888, but regarding the following variables:

• _governanceCall in Governor
• executionId in AccessManager

The Governor call is a Bytes32Deque, which makes me wonder if we may add a transient version of some libraries.
Both are technically breaking changes, but I don't see how they would affect the contract given the values are private.

[Amxx]

For structs like Bytes32Deque, I would wait for solidity to support the transient keyword, and then I would duplicate all function that currently have a storage pointer in their argument to also have a transient version of the function.

Like this