This is an archive of the discontinued LLVM Phabricator instance.

Differential D145652

[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
Herald added a subscriber: libcxx-commits. · View Herald Transcript
Harbormaster completed remote builds in B218297: Diff 503626.Mar 8 2023, 10:26 PM
Joy12138 mentioned this in D144362: [libcxx] <experimental/simd> Add ABI tags, class template simd/simd_mask implementations. Add related simd traits and tests..Mar 8 2023, 10:27 PM
Joy12138 added a child revision: D144362: [libcxx] <experimental/simd> Add ABI tags, class template simd/simd_mask implementations. Add related simd traits and tests..
Joy12138 updated this revision to Diff 503634.Mar 8 2023, 10:41 PM

Fix descriptions

Harbormaster completed remote builds in B218303: Diff 503634.Mar 8 2023, 11:07 PM
Joy12138 updated this revision to Diff 503668.Mar 9 2023, 12:26 AM

Fix paper title overline

Harbormaster completed remote builds in B218327: Diff 503668.Mar 9 2023, 1:23 AM
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

Herald added a subscriber: arphaman. · View Herald TranscriptMar 9 2023, 10:11 PM
Joy12138 marked 3 inline comments as done.Mar 9 2023, 10:14 PM
Harbormaster completed remote builds in B218592: Diff 504027.Mar 9 2023, 11:44 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？

philnik added a comment.Mar 13 2023, 5:10 AM
In D145652#4187965, @Joy12138 wrote:

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
Closed by commit rG9d8b60d08ab9: [libcxx] Add status paper for Parallelism-TS (authored by Joy12138). · Explain Why
This revision was automatically updated to reflect the committed changes.
Joy12138 added a commit: rG9d8b60d08ab9: [libcxx] Add status paper for Parallelism-TS.
Herald added a subscriber: mikhail.ramalho. · View Herald TranscriptMar 16 2023, 9:31 PM
philnik removed a child revision: D144362: [libcxx] <experimental/simd> Add ABI tags, class template simd/simd_mask implementations. Add related simd traits and tests..Jun 23 2023, 6:40 PM

Revision Contents

PathSize
libcxx/
docs/
Status/
29 lines
12 lines

Diff 503626

libcxx/docs/Status/Parallelism.rst

  • This file was added.
.. parallelism-status:
philnikUnsubmitted
Done
ReplyInline Actions

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

philnik: You also have to add a new entry in the list at `libcxx/docs/index.rst:37`. This should fix the…
===========================
libc++ Parallelism Status (N4808)
===========================
philnikUnsubmitted
Done
ReplyInline Actions
=================================
- libc++ Parallelism Status (N4808)
+ libc++ Parallelism TS Status (N4808)
=================================
philnik:
.. include:: ../Helpers/Styles.rst
.. contents::
:local:
Overview
========
This document contains the status of the Parallelism implementation in libc++.
It is used to track both the status of the sub-projects of the effort and who
philnikUnsubmitted
Done
ReplyInline Actions
========
- This document contains the status of the Parallelism implementation in libc++.
+ This document contains the status of the Parallelism TS implementation in libc++.
It is used to track both the status of the sub-projects of the effort and who
philnik:
is assigned to these sub-projects. This avoids duplicating effort.
If you are interested in contributing to this effort, please send a message
to the #libcxx channel in the LLVM discord. Please *do not* start working
on any items below that has already been assigned to someone else.
Sub-projects in the Implementation Effort
=========================================
.. csv-table::
:file: ParallelismProjects.csv
:header-rows: 1
:widths: auto

libcxx/docs/Status/ParallelismProjects.csv

  • This file was added.
Section,Description,Dependencies,Assignee,Complete
| `[parallel.exceptions] <https://wg21.link/N4808>`_, "Parallel Exceptions", None, unassigned, |Not Started|
| `[parallel.execpol] <https://wg21.link/N4808>`_, "Execution policies", None, unassigned, |Not Started|
| `[parallel.alg] <https://wg21.link/N4808>`_, "Parallel algorithms", None, unassigned, |Not Started|
| `[parallel.taskblock] <https://wg21.link/N4808>`_, "Task Block", None, unassigned, |Not Started|
| `[parallel.simd.abi] <https://wg21.link/N4808>`_, "simd ABI tags", None, Yin Zhang, |In Progress|
| `[parallel.simd.traits] <https://wg21.link/N4808>`_, "simd type traits", None, Yin Zhang, |In Progress|
| `[parallel.simd.whereexpr] <https://wg21.link/N4808>`_, "Where expression class templates", None, Yin Zhang, |In Progress|
| `[parallel.simd.class] <https://wg21.link/N4808>`_, "Class template simd", None, Yin Zhang, |In Progress|
| `[parallel.simd.nonmembers] <https://wg21.link/N4808>`_, "simd non-member operations", None, Yin Zhang, |In Progress|
| `[parallel.simd.mask.class] <https://wg21.link/N4808>`_, "Reductions", None, Yin Zhang, |In Progress|
| `[parallel.simd.mask.nonmembers] <https://wg21.link/N4808>`_, "Non-member operations", None, Yin Zhang, |In Progress|