[PATCH] D13604: Fix to allow C conversions on arguments passed to functions with __attribute__((overloadable)) in C

Richard Smith via cfe-commits cfe-commits at lists.llvm.org
Fri Oct 9 15:21:30 PDT 2015


rsmith accepted this revision.
rsmith added a comment.
This revision is now accepted and ready to land.

Instead of `FixRHS`, I'd prefer either `ConvertRHS` or `PerformConversion` as your parameter name. Otherwise, LGTM, thanks!


================
Comment at: lib/Sema/SemaOverload.cpp:1444-1445
@@ -1438,3 +1443,4 @@
+  // abort early. When overloading in C, however, we do permit them.
   if (FromType->isRecordType() || ToType->isRecordType()) {
     if (S.getLangOpts().CPlusPlus)
       return false;
----------------
Fold these two 'if's together.


http://reviews.llvm.org/D13604





More information about the cfe-commits mailing list