[clang] [clang] Simplify the overload resolution logic for operator new and new[] (PR #211482)

[Oliver Hunt via cfe-commits]

================
@@ -57,3 +56,17 @@ void f() {
   TestType *t = new TestType;
   delete t;
 }
+
+#if !defined(INVALID_TYPE_IDENTITY_VERSION)
+struct Bad {};
+template <> struct std::type_identity<Bad>; // #incomplete_specialization
+
+// This is a pure implementation test to ensure correct caching behavior if
+// constructing the type_identity argument fails.
+void failedTypeIdentitySpecialization() {
----------------
ojhunt wrote:

to clarify: not saying this is the best approach, I think your consistency argument is strong so I will change it

https://github.com/llvm/llvm-project/pull/211482