Page MenuHomePhabricator

Add test for spaceship operator to __config

Authored by davezarzycki on Sat, Jan 18, 11:14 AM.



The libcxx test suite auto-detects spaceship operator, but __config does not. This means that the libcxx test suite has been broken for over a month when using top-of-tree clang. This also really ought to be fixed before 10.0.

See: bc633a42dd409dbeb456263e3388b8caa4680aa0

Diff Detail

Event Timeline

davezarzycki created this revision.Sat, Jan 18, 11:14 AM
Herald added a project: Restricted Project. · View Herald Transcript
davezarzycki edited the summary of this revision. (Show Details)Sat, Jan 18, 11:22 AM
davezarzycki edited the summary of this revision. (Show Details)Sat, Jan 18, 11:48 AM

Ping. Feedback would be appreciated. Thanks

hans added a subscriber: hans.Thu, Jan 23, 12:21 PM

Eric, Marshall, what do you think?

I'm seeing these failures locally to, on the 10.x branch, and they also seem to be failing on the pre-submit bot, e.g.

I've also been experiencing these test failures and I'm very much interested in this getting landed.

(I've done this very change independently from you in D73351, so LGTM, but I am not a libc++ expert though.)

EricWF accepted this revision.Fri, Jan 24, 10:22 AM


This revision is now accepted and ready to land.Fri, Jan 24, 10:22 AM
hans added a comment.Fri, Jan 24, 10:38 AM

Thanks! Cherry-picked to 10.x in 39c349e8fc7f4b334cf4b30724b28dfce44a024e
Please let me know if there are any follow-ups.