EIP712Upgradeable.sol

[songhobby]

_EIP712NameHash and _EIP712VersionHash don't cache the hashes.

💻 Environment

EIP712Upgradeable.sol

Suggestions
Since this is an upgradeable contract, _EIP712NameHash and _EIP712VersionHash should simply return the hashes as byte32 literals or constants.
To ensure correctness, the equality of the _nameHash == keccak256(_name) can be checked in the initializer or use inline hash computation to initialize the constant.
With that being said, the storage slot is not needed in EIP712Upgradeable contract.

    string private constant _name = "abc";
    string private constant _version = "1";
    byte32 private constant _nameHash = keccak256("abc");
    byte32 private constant _version = keccak256("1");