This is an archive of the discontinued LLVM Phabricator instance.

[libcxx] Add status paper for Parallelism-TS
ClosedPublic

Authored by Joy12138 on Mar 8 2023, 10:21 PM.

Details

Reviewers
philnik
Mordante
Group Reviewers
Restricted Project
Commits
rG9d8b60d08ab9: [libcxx] Add status paper for Parallelism-TS

Diff Detail

Event Timeline

Joy12138 created this revision.Mar 8 2023, 10:21 PM
Herald added a project: Restricted Project. · View Herald TranscriptMar 8 2023, 10:21 PM
Joy12138 requested review of this revision.Mar 8 2023, 10:21 PM
Herald added a project: Restricted Project. · View Herald TranscriptMar 8 2023, 10:21 PM
Herald added a reviewer: Restricted Project. · View Herald Transcript
Joy12138 updated this revision to Diff 503634.Mar 8 2023, 10:41 PM

Fix descriptions

Joy12138 updated this revision to Diff 503668.Mar 9 2023, 12:26 AM

Fix paper title overline

philnik added a subscriber: philnik.Mar 9 2023, 4:34 AM
philnik added inline comments.
libcxx/docs/Status/Parallelism.rst
2

You also have to add a new entry in the list at libcxx/docs/index.rst:37. This should fix the CI.

5
16
Joy12138 updated this revision to Diff 504027.Mar 9 2023, 10:11 PM

Fix CI and title

Joy12138 marked 3 inline comments as done.Mar 9 2023, 10:14 PM
philnik accepted this revision.Mar 10 2023, 1:56 AM
This revision is now accepted and ready to land.Mar 10 2023, 1:56 AM
Mordante accepted this revision.Mar 10 2023, 9:58 AM
Mordante added a subscriber: Mordante.

Thanks! LGTM!

Joy12138 added a comment.EditedMar 12 2023, 9:56 PM

Should I land this patch now or wait for the others?

In addition, do the failed Windows CIs matter?

Should I land this patch now or wait for the others?

This should be fine to land now.

In addition, do the failed Windows CIs matter?

No, these are unrelated.

This revision was landed with ongoing or failed builds.Mar 16 2023, 9:31 PM
This revision was automatically updated to reflect the committed changes.