[PATCH] D55045: Add a version of std::function that includes a few optimizations.

Louis Dionne via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 29 09:03:41 PST 2018


ldionne requested changes to this revision.
ldionne added a comment.
This revision now requires changes to proceed.

I'd like to see benchmarking results that show the benefit of this approach.

Also, it's not clear to me that trying to "merge" the two implementations is the right choice. Maybe we want a straight up different implementation (i.e. replace `std::function` as a whole instead of having many `#ifdef`s in the existing `std::function`).


Repository:
  rCXX libc++

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D55045/new/

https://reviews.llvm.org/D55045





More information about the llvm-commits mailing list