[PATCH] D58145: [Sema] Fix a bogus -Wconversion warning
Nick Desaulniers via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Feb 12 16:40:00 PST 2019
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
More information about the cfe-commits
mailing list