This is an archive of the discontinued LLVM Phabricator instance.

[libcxx] Add Apple blocks support to std::function
AbandonedPublic

Authored by EricWF on Oct 2 2014, 8:41 PM.

Details

Reviewers
None

Diff Detail

Event Timeline

EricWF updated this revision to Diff 14367.Oct 2 2014, 8:41 PM
EricWF retitled this revision from to [libcxx] Add Apple blocks support to std::function.
EricWF updated this object.
EricWF edited the test plan for this revision. (Show Details)
EricWF added inline comments.Oct 2 2014, 8:56 PM
include/functional
1431

I agree this should never be called. I'm unsure of this use of _LIBCPP_ASSERT because (as I understand it) _LIBCPP_ASSERT should be used to verify that the user is using libc++ correctly and not that our implementation is correct.

1446–1448

When is it possible for __f_ to be null?

EricWF abandoned this revision.Oct 18 2014, 4:32 PM