[PATCH] Follow-up to D6217

Anton Korobeynikov anton at korobeynikov.info
Thu Nov 13 20:12:08 PST 2014


================
Comment at: lib/Sema/SemaChecking.cpp:3436-3444
@@ +3435,11 @@
+
+    if (TrueTy == FalseTy)
+      return { TrueTy, TrueName };
+    else if (TrueTy.isNull())
+      return { FalseTy, FalseName };
+    else if (FalseTy.isNull())
+      return { TrueTy, TrueName };
+  }
+
+  return { QualType(), StringRef() };
+}
----------------
rsmith wrote:
> I would be surprised if all our supported compilers cope with this syntax. Happily surprised, but surprised nonetheless.
If MSVC will have problems here, I will fall back to std::make_pair here surely.

http://reviews.llvm.org/D6256






More information about the cfe-commits mailing list