display:none;
elements are not in the render tree all, so they will perform better at face value.
I doubt you will have any real visible performance problems from this though. If you need opacity: 0
or visibility: hidden
because of their functionality, then just use them. If you don’t need the functionality, then use display: none;