[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