Page MenuHomePhabricator

Please use GitHub pull requests for new patches. Avoid migrating existing patches. Phabricator shutdown timeline

mumbleskates (Kent Ross)
User

Projects

User does not belong to any projects.

User Details

User Since
Aug 3 2021, 11:43 PM (122 w, 2 d)

Recent Activity

Oct 17 2022

mumbleskates added a comment to D136050: [libc++] Fix missing requires clause on variant operator<=>.

thanks! LGTM. i must have been looking at a reference that's missing that requires...

Oct 17 2022, 4:40 PM · Restricted Project, Restricted Project

Aug 25 2022

mumbleskates added inline comments to D132284: [libc++] Reduces the number of transitive includes..
Aug 25 2022, 11:33 AM · Restricted Project, Restricted Project

Aug 21 2022

mumbleskates added inline comments to D131395: [libc++] Implement `lexicographical_compare_three_way`.
Aug 21 2022, 12:52 PM · Restricted Project, Restricted Project

Aug 20 2022

mumbleskates added a comment to D132312: [libc++][spaceship] Implement `operator<=>` for `list`.

One thing I have thought about in our march towards finishing the lexically ordered containers is that we should be able to write a test utils header with one or more templated test functions that will work for most or all of these containers. We can possibly even add (in another header?) some helpful types like weakly ordered ints, types that compare with user orderings, etc. so we don't have to keep writing them over and over. This feels fairly sane to me but I don't know what the general vibe is on how DRY is too DRY for our test suite, and maybe it's better to just copy-paste those parts of the code each time.

Aug 20 2022, 8:21 PM · Restricted Project, Restricted Project
mumbleskates added inline comments to D131928: [libcxx][spaceship][doc] Repair links and clean up spaceship progress doc.
Aug 20 2022, 3:01 PM · Restricted Project, Restricted Project
mumbleskates added inline comments to D131395: [libc++] Implement `lexicographical_compare_three_way`.
Aug 20 2022, 2:19 PM · Restricted Project, Restricted Project
mumbleskates added inline comments to D131395: [libc++] Implement `lexicographical_compare_three_way`.
Aug 20 2022, 1:51 AM · Restricted Project, Restricted Project

Aug 17 2022

mumbleskates added inline comments to D131928: [libcxx][spaceship][doc] Repair links and clean up spaceship progress doc.
Aug 17 2022, 12:16 PM · Restricted Project, Restricted Project
mumbleskates committed rG081dad9e9e8d: [libcxx][spaceship][doc] Repair links and clean up spaceship progress doc (authored by mumbleskates).
[libcxx][spaceship][doc] Repair links and clean up spaceship progress doc
Aug 17 2022, 12:15 PM · Restricted Project, Restricted Project
mumbleskates closed D131928: [libcxx][spaceship][doc] Repair links and clean up spaceship progress doc.
Aug 17 2022, 12:15 PM · Restricted Project, Restricted Project
mumbleskates updated the summary of D131928: [libcxx][spaceship][doc] Repair links and clean up spaceship progress doc.
Aug 17 2022, 12:13 PM · Restricted Project, Restricted Project

Aug 16 2022

mumbleskates added inline comments to D131395: [libc++] Implement `lexicographical_compare_three_way`.
Aug 16 2022, 8:24 PM · Restricted Project, Restricted Project
mumbleskates removed reviewers for D131928: [libcxx][spaceship][doc] Repair links and clean up spaceship progress doc: lebedev.ri, jhenderson, antiagainst, nicolasvasilache, herhut, rriddle, aartbik, MaskRay, sscalpone, sjarus, clementval, bondhugula, NoQ, dcaballe, JDevlieghere, jdoerfert, aaron.ballman, rafauler, Amir, maksfb.
Aug 16 2022, 7:52 PM · Restricted Project, Restricted Project
mumbleskates added a comment to D131928: [libcxx][spaceship][doc] Repair links and clean up spaceship progress doc.

Sorry! Something has gone horribly wrong with my usage of arcanist and it diffed against a very old version of main. i will try to repair the reviewers list.

Aug 16 2022, 7:47 PM · Restricted Project, Restricted Project
mumbleskates updated the diff for D131928: [libcxx][spaceship][doc] Repair links and clean up spaceship progress doc.
  • Merge branch 'main' into doc2
Aug 16 2022, 7:46 PM · Restricted Project, Restricted Project
mumbleskates updated the diff for D131928: [libcxx][spaceship][doc] Repair links and clean up spaceship progress doc.
  • Merge remote-tracking branch 'upstream/main' into doc2
Aug 16 2022, 7:43 PM · Restricted Project, Restricted Project
mumbleskates updated the diff for D131928: [libcxx][spaceship][doc] Repair links and clean up spaceship progress doc.
  • Revert "use a version of the link that does not currently 404 on wg21.link"
Aug 16 2022, 7:42 PM · Restricted Project, Restricted Project

Aug 15 2022

mumbleskates retitled D131928: [libcxx][spaceship][doc] Repair links and clean up spaceship progress doc from [libcxx][spaceship][doc] Repair links to the optional container requirements for operator<=> to [libcxx][spaceship][doc] Repair links and clean up spaceship progress doc.
Aug 15 2022, 6:59 PM · Restricted Project, Restricted Project
mumbleskates updated the diff for D131928: [libcxx][spaceship][doc] Repair links and clean up spaceship progress doc.
  • fix typo in "chrono::time_point"
  • fix broken formatting for the "basic_string" hyperlink
  • link to differentials of completed project items
  • Merge branch 'main' into doc2
Aug 15 2022, 6:58 PM · Restricted Project, Restricted Project
mumbleskates added inline comments to D131928: [libcxx][spaceship][doc] Repair links and clean up spaceship progress doc.
Aug 15 2022, 4:27 PM · Restricted Project, Restricted Project
mumbleskates updated the diff for D131928: [libcxx][spaceship][doc] Repair links and clean up spaceship progress doc.
  • use a version of the link that does not currently 404 on wg21.link
Aug 15 2022, 4:21 PM · Restricted Project, Restricted Project
mumbleskates requested review of D131928: [libcxx][spaceship][doc] Repair links and clean up spaceship progress doc.
Aug 15 2022, 4:16 PM · Restricted Project, Restricted Project

Aug 14 2022

mumbleskates committed rGc4566cac490c: [libc++][spaceship] Implement std::variant::operator<=> (authored by mumbleskates).
[libc++][spaceship] Implement std::variant::operator<=>
Aug 14 2022, 4:17 PM · Restricted Project, Restricted Project
mumbleskates closed D131372: [libc++][spaceship] Implement std::variant::operator<=>.
Aug 14 2022, 4:17 PM · Restricted Project, Restricted Project

Aug 13 2022

mumbleskates added a comment to D131372: [libc++][spaceship] Implement std::variant::operator<=>.

okay, every comment is either resolved or involves work that should go into another differential.

Aug 13 2022, 5:23 PM · Restricted Project, Restricted Project
mumbleskates added inline comments to D131372: [libc++][spaceship] Implement std::variant::operator<=>.
Aug 13 2022, 5:19 PM · Restricted Project, Restricted Project

Aug 12 2022

mumbleskates added a comment to D131372: [libc++][spaceship] Implement std::variant::operator<=>.

i suspect we can ship this differential as-is without fiddling with test_comparisons.h any further, and any further changes we want to make to that file should be done in another differential.

Aug 12 2022, 12:58 PM · Restricted Project, Restricted Project
mumbleskates updated the diff for D131372: [libc++][spaceship] Implement std::variant::operator<=>.
  • re-add possibly less confusing assert to testComparisons that does not reference a parameter it doesn't have
  • Merge branch 'main' into variant
  • add sfinae tests for variant::operator<=>
  • variant::operator<=> does NOT require three_way_comparable for its types in the standard
Aug 12 2022, 12:54 PM · Restricted Project, Restricted Project
mumbleskates added inline comments to D131395: [libc++] Implement `lexicographical_compare_three_way`.
Aug 12 2022, 11:21 AM · Restricted Project, Restricted Project

Aug 11 2022

mumbleskates added inline comments to D131372: [libc++][spaceship] Implement std::variant::operator<=>.
Aug 11 2022, 11:31 PM · Restricted Project, Restricted Project
mumbleskates added inline comments to D131372: [libc++][spaceship] Implement std::variant::operator<=>.
Aug 11 2022, 3:33 PM · Restricted Project, Restricted Project
mumbleskates updated the diff for D131372: [libc++][spaceship] Implement std::variant::operator<=>.
  • simplify TEST_NODISCARD definition per feedback
Aug 11 2022, 3:33 PM · Restricted Project, Restricted Project
mumbleskates added inline comments to D131372: [libc++][spaceship] Implement std::variant::operator<=>.
Aug 11 2022, 1:27 PM · Restricted Project, Restricted Project
mumbleskates updated the diff for D131372: [libc++][spaceship] Implement std::variant::operator<=>.
  • assert return value types and simplify order value expressions
  • add TEST_NODISCARD macro
  • use TEST_NODISCARD
  • formatting
Aug 11 2022, 1:23 PM · Restricted Project, Restricted Project

Aug 10 2022

mumbleskates added a comment to D131395: [libc++] Implement `lexicographical_compare_three_way`.

I think the random access iterator optimization is probably pretty important here. it may be doable to eventually derive this optimization from very simple iterator types like int*, but there are plenty of examples of very non-trivial iterator types where this may not be the case. iterators over order-statistic trees can satisfy every requirement of random_access_iterator while being somewhat complex.

Aug 10 2022, 10:36 PM · Restricted Project, Restricted Project
mumbleskates added inline comments to D130295: [libc++] Uses operator<=> in string_view.
Aug 10 2022, 8:32 PM · Restricted Project, Restricted Project
mumbleskates added inline comments to D131372: [libc++][spaceship] Implement std::variant::operator<=>.
Aug 10 2022, 7:31 PM · Restricted Project, Restricted Project
mumbleskates updated the diff for D131372: [libc++][spaceship] Implement std::variant::operator<=>.
  • correctly wrap testOrder and testComparisons in asserts
  • fix nits
  • run git clang-format on all changes
  • manually clang-format the added function
  • manually fix spacing of multiline macros
  • add partial_ordering::unordered test cases & utils support
  • refactor testComparisons and clean up includes
  • add nodiscard vis a vis D131364
  • order things sensibly because this is C++
Aug 10 2022, 7:31 PM · Restricted Project, Restricted Project
mumbleskates accepted D131364: [libc++][test] Mark `test_comparisons.h` helpers as nodiscard.

Thanks! looks good to me

Aug 10 2022, 5:33 PM · Restricted Project, Restricted Project

Aug 9 2022

mumbleskates updated the diff for D131372: [libc++][spaceship] Implement std::variant::operator<=>.
  • also use _LIBCPP_HIDE_FROM_ABI in monostate
Aug 9 2022, 1:32 PM · Restricted Project, Restricted Project
mumbleskates added inline comments to D131372: [libc++][spaceship] Implement std::variant::operator<=>.
Aug 9 2022, 1:30 PM · Restricted Project, Restricted Project
mumbleskates updated the diff for D131372: [libc++][spaceship] Implement std::variant::operator<=>.
  • modernize variant three_way test
  • _LIBCPP_INLINE_VISIBILITY --> _LIBCPP_HIDE_FROM_ABI, remove unnecessary inlines
Aug 9 2022, 1:30 PM · Restricted Project, Restricted Project

Aug 8 2022

mumbleskates retitled D131372: [libc++][spaceship] Implement std::variant::operator<=> from Implement std::variant::operator<=> to [libc++][spaceship] Implement std::variant::operator<=>.
Aug 8 2022, 2:08 PM · Restricted Project, Restricted Project
mumbleskates added inline comments to D131372: [libc++][spaceship] Implement std::variant::operator<=>.
Aug 8 2022, 2:06 PM · Restricted Project, Restricted Project
mumbleskates added inline comments to D131372: [libc++][spaceship] Implement std::variant::operator<=>.
Aug 8 2022, 1:48 PM · Restricted Project, Restricted Project
mumbleskates updated the diff for D131372: [libc++][spaceship] Implement std::variant::operator<=>.
  • add comment in test for tested operator
  • don't conditionally include <compare>
  • use runtime-and-static assert pattern for monostate test
  • use test macros for monostate relops
Aug 8 2022, 1:48 PM · Restricted Project, Restricted Project
mumbleskates added inline comments to D131363: [libc++] Implement `operator<=>` for `error_category`.
Aug 8 2022, 1:42 PM · Restricted Project, Restricted Project
mumbleskates accepted D131419: [NFC][libc++][test] Removes unneeded code..

Thanks for looking!

Aug 8 2022, 11:11 AM · Restricted Project, Restricted Project

Aug 7 2022

mumbleskates added inline comments to D131363: [libc++] Implement `operator<=>` for `error_category`.
Aug 7 2022, 8:11 PM · Restricted Project, Restricted Project
mumbleskates added inline comments to D131363: [libc++] Implement `operator<=>` for `error_category`.
Aug 7 2022, 8:10 PM · Restricted Project, Restricted Project
mumbleskates updated the diff for D131372: [libc++][spaceship] Implement std::variant::operator<=>.
  • add missing include for monostate
Aug 7 2022, 7:43 PM · Restricted Project, Restricted Project
mumbleskates added a comment to D131372: [libc++][spaceship] Implement std::variant::operator<=>.

Updated SpaceshipProjects and added the missing monostate operator as well

Aug 7 2022, 7:24 PM · Restricted Project, Restricted Project
mumbleskates updated the diff for D131372: [libc++][spaceship] Implement std::variant::operator<=>.
  • delete dead test function
  • use a result type alias
  • also implement std::monostate::operator<=>
  • update spaceship projects
Aug 7 2022, 7:23 PM · Restricted Project, Restricted Project
mumbleskates requested review of D131372: [libc++][spaceship] Implement std::variant::operator<=>.
Aug 7 2022, 5:33 PM · Restricted Project, Restricted Project
mumbleskates added inline comments to D130295: [libc++] Uses operator<=> in string_view.
Aug 7 2022, 4:15 PM · Restricted Project, Restricted Project

Aug 6 2022

mumbleskates commandeered D80891: [libcxx] adds consistent comparison for `basic_string_view`.

Feeling like getting something done this weekend

Aug 6 2022, 2:30 PM · Restricted Project, Restricted Project

Jan 25 2022

mumbleskates added a comment to D62641: Support arrays in make_shared and allocate_shared (P0674R1).

What's the status of this one? Looks stalled, and I understand it's still a blocker for make_{unique,shared}_for_overwrite in D90111

Jan 25 2022, 12:34 AM · Restricted Project, Restricted Project

Nov 30 2021

mumbleskates committed rG7d18267cfef7: [libcxx][doc] Document recent spaceship projects progress (authored by mumbleskates).
[libcxx][doc] Document recent spaceship projects progress
Nov 30 2021, 6:47 PM
mumbleskates closed D114682: [libcxx][doc] Document recent spaceship projects progress.
Nov 30 2021, 6:47 PM · Restricted Project
mumbleskates added inline comments to D114682: [libcxx][doc] Document recent spaceship projects progress.
Nov 30 2021, 12:41 PM · Restricted Project
mumbleskates updated the diff for D114682: [libcxx][doc] Document recent spaceship projects progress.
  • Revert "add reference to P1614 in the spaceship RST"
Nov 30 2021, 12:41 PM · Restricted Project
mumbleskates updated the diff for D114682: [libcxx][doc] Document recent spaceship projects progress.
  • correct section name for move_iterator
Nov 30 2021, 11:44 AM · Restricted Project
mumbleskates updated the diff for D114682: [libcxx][doc] Document recent spaceship projects progress.
  • Revert "move [charconv.syn] to Formatting"
  • add reference to P1614 in the spaceship RST
Nov 30 2021, 11:33 AM · Restricted Project

Nov 28 2021

mumbleskates edited reviewers for D114682: [libcxx][doc] Document recent spaceship projects progress, added: Quuxplusone; removed: ldionne.
Nov 28 2021, 6:46 PM · Restricted Project
mumbleskates added a reviewer for D114682: [libcxx][doc] Document recent spaceship projects progress: Mordante.
Nov 28 2021, 6:38 PM · Restricted Project
mumbleskates updated the diff for D114682: [libcxx][doc] Document recent spaceship projects progress.
  • add link to coroutine differential
  • move [charconv.syn] to Formatting
Nov 28 2021, 6:37 PM · Restricted Project
mumbleskates requested review of D114682: [libcxx][doc] Document recent spaceship projects progress.
Nov 28 2021, 6:28 PM · Restricted Project

Nov 18 2021

mumbleskates committed rG870dfa6b08d0: [libc++][doc] Mark project for [cmp.concept] done (authored by mumbleskates).
[libc++][doc] Mark project for [cmp.concept] done
Nov 18 2021, 8:16 PM
mumbleskates closed D114203: [libc++][doc] Mark project for [cmp.concept] done.
Nov 18 2021, 8:16 PM · Restricted Project
mumbleskates requested review of D114203: [libc++][doc] Mark project for [cmp.concept] done.
Nov 18 2021, 4:07 PM · Restricted Project

Oct 12 2021

mumbleskates added inline comments to D110738: [libc++] [P1614] Implement the first half of [cmp.alg]: strong_order, weak_order, partial_order..
Oct 12 2021, 10:21 AM · Restricted Project

Oct 11 2021

mumbleskates added inline comments to D110738: [libc++] [P1614] Implement the first half of [cmp.alg]: strong_order, weak_order, partial_order..
Oct 11 2021, 6:22 PM · Restricted Project
mumbleskates added inline comments to D110738: [libc++] [P1614] Implement the first half of [cmp.alg]: strong_order, weak_order, partial_order..
Oct 11 2021, 2:14 PM · Restricted Project
mumbleskates added inline comments to D110738: [libc++] [P1614] Implement the first half of [cmp.alg]: strong_order, weak_order, partial_order..
Oct 11 2021, 1:55 PM · Restricted Project

Oct 10 2021

mumbleskates added inline comments to D111458: [libc++][doc] Use issue labels..
Oct 10 2021, 4:34 PM · Restricted Project
mumbleskates added a comment to D111458: [libc++][doc] Use issue labels..

This seems like a good ideal to work towards to me, as the organization of these issues is a bit messy.

Oct 10 2021, 4:33 PM · Restricted Project
mumbleskates added inline comments to D111458: [libc++][doc] Use issue labels..
Oct 10 2021, 9:54 AM · Restricted Project

Oct 8 2021

mumbleskates committed rGb80f2dfd11e2: [libc++][spaceship] Implement std::tuple::operator<=> (authored by mumbleskates).
[libc++][spaceship] Implement std::tuple::operator<=>
Oct 8 2021, 4:30 PM
mumbleskates closed D108250: [libc++][spaceship] Implement std::tuple::operator<=>.
Oct 8 2021, 4:30 PM · Restricted Project
mumbleskates updated the diff for D108250: [libc++][spaceship] Implement std::tuple::operator<=>.
  • add missing comments
Oct 8 2021, 2:06 PM · Restricted Project
mumbleskates updated the diff for D108250: [libc++][spaceship] Implement std::tuple::operator<=>.
  • also test size-incompatible < in all versions
Oct 8 2021, 12:42 PM · Restricted Project

Oct 7 2021

mumbleskates updated the diff for D108250: [libc++][spaceship] Implement std::tuple::operator<=>.
  • reduce the scope of incompatible-size tuple comparison tests
Oct 7 2021, 7:30 PM · Restricted Project

Oct 6 2021

mumbleskates added inline comments to D108250: [libc++][spaceship] Implement std::tuple::operator<=>.
Oct 6 2021, 8:23 PM · Restricted Project

Sep 29 2021

mumbleskates updated the diff for D108250: [libc++][spaceship] Implement std::tuple::operator<=>.
  • rebase
Sep 29 2021, 9:41 PM · Restricted Project

Sep 24 2021

mumbleskates added inline comments to D108250: [libc++][spaceship] Implement std::tuple::operator<=>.
Sep 24 2021, 3:44 PM · Restricted Project
mumbleskates added inline comments to D108250: [libc++][spaceship] Implement std::tuple::operator<=>.
Sep 24 2021, 3:19 PM · Restricted Project
mumbleskates updated the diff for D108250: [libc++][spaceship] Implement std::tuple::operator<=>.
  • work around a GCC segfault bug
Sep 24 2021, 2:52 AM · Restricted Project
mumbleskates added inline comments to D108250: [libc++][spaceship] Implement std::tuple::operator<=>.
Sep 24 2021, 1:17 AM · Restricted Project
mumbleskates added inline comments to D108250: [libc++][spaceship] Implement std::tuple::operator<=>.
Sep 24 2021, 12:38 AM · Restricted Project

Sep 23 2021

mumbleskates updated the diff for D108250: [libc++][spaceship] Implement std::tuple::operator<=>.
  • move and standardize the test for synth-three-way
Sep 23 2021, 11:30 PM · Restricted Project
mumbleskates added inline comments to D107721: [libc++][spaceship] Implement std::pair::operator<=>.
Sep 23 2021, 9:02 PM · Restricted Project

Sep 22 2021

mumbleskates committed rGf4abdb0c074b: [libc++][spaceship] Implement std::pair::operator<=> (authored by mumbleskates).
[libc++][spaceship] Implement std::pair::operator<=>
Sep 22 2021, 10:40 PM
mumbleskates closed D107721: [libc++][spaceship] Implement std::pair::operator<=>.
Sep 22 2021, 10:40 PM · Restricted Project
mumbleskates added inline comments to D107721: [libc++][spaceship] Implement std::pair::operator<=>.
Sep 22 2021, 7:34 PM · Restricted Project
mumbleskates updated the diff for D108250: [libc++][spaceship] Implement std::tuple::operator<=>.
  • don't cast-to-void at top-level scope
  • adl-proofing
  • address comments
Sep 22 2021, 7:34 PM · Restricted Project
mumbleskates updated the diff for D107721: [libc++][spaceship] Implement std::pair::operator<=>.
  • address comments
Sep 22 2021, 7:34 PM · Restricted Project
mumbleskates added inline comments to D107721: [libc++][spaceship] Implement std::pair::operator<=>.
Sep 22 2021, 3:04 PM · Restricted Project
mumbleskates added inline comments to D108250: [libc++][spaceship] Implement std::tuple::operator<=>.
Sep 22 2021, 2:09 PM · Restricted Project
mumbleskates added inline comments to D107721: [libc++][spaceship] Implement std::pair::operator<=>.
Sep 22 2021, 1:57 PM · Restricted Project
mumbleskates added inline comments to D107721: [libc++][spaceship] Implement std::pair::operator<=>.
Sep 22 2021, 1:56 PM · Restricted Project