[PATCH] D15174: [MSVC] Fix for http://llvm.org/PR25636: indexed accessor property not supported correctly.

John McCall via cfe-commits cfe-commits at lists.llvm.org
Wed Dec 9 08:53:55 PST 2015


rjmccall added a comment.

Thanks!  Just a slight tweak to the comment, then LGTM.


================
Comment at: lib/Sema/SemaPseudoObject.cpp:259
@@ +258,3 @@
+    ///
+    /// If this method returns false, and the set value isn't capturable for
+    /// some reason, the result of the expression will be void.
----------------
"If this method returns *true*", since you picked this one.


http://reviews.llvm.org/D15174





More information about the cfe-commits mailing list