Diagnostic suppression checking in SuppressMessageAttributeState may do unnecessary work

See https://github.com/dotnet/roslyn/pull/51792

If both the definition of an attribute is missing and the definition of (Unconditional)SuppressMessageAttribute is missing, the implementation may do extra work to try to suppress a message, when it could avoid it.