nickdesaulniers accepted this revision. nickdesaulniers added a comment. This revision is now accepted and ready to land.
Thanks for fixing this regression (sorry, I caused it). Doesn't regress the tests I added. You'll probably want to request to Hans that this gets picked up for the 8.0 release. Not sure if @rsmith has comments on the added `fixme`? ================ Comment at: clang/test/SemaObjC/conversion.m:23 + double local = 42; + dp.d += local; +} ---------------- does this need a `// no-warning` comment? Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D58145/new/ https://reviews.llvm.org/D58145 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
