Revert "Avoid hanging trying to diff Data and similar collections. (#1644)"

[stmontgomery]

This reverts the change recently made in #1644. It ended up causing slow type-checking of certain complex expressions passed to #expect, such as this one in particular in the swift-build project:

#expect(output.unsafeStringValue.split(separator: "\n").map({ $0.split("=").0 }).sorted().filter { $0 != "VCToolsInstallDir" && !$0.hasPrefix("ANDROID_") && !$0.hasPrefix("QNX_") } == [
    "LLBUILD_BUILD_ID", "LLBUILD_CONTROL_FD", "LLBUILD_LANE_ID", "LLBUILD_TASK_ID"
])

It appears the new __checkBinaryOperation() overload added in that PR caused type-checking to become more complex and slower for complex expressions such as this. It may be possible to implement an alternate approach for this which checks the type at runtime only, to avoid needing an additional overload.

Related to: rdar://173002947

Checklist:

• Code and documentation should follow the style of the Style Guide.
• If public symbols are renamed or modified, DocC references should be updated.