llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-clang
Author: Chi-Chun, Chen (chichunchen)
<details>
<summary>Changes</summary>
Correct the colors used in the OpenMP support tables so they consistently match
their status text:
- :good: (green) is for 'done' only
- :part: (yellow) is for in-progress states ('partial', 'worked on', 'in
progress', 'prototyped', etc.)
- :none: (red) is for 'unclaimed' only
Assisted with copilot
---
Full diff: https://github.com/llvm/llvm-project/pull/189050.diff
1 Files Affected:
- (modified) clang/docs/OpenMPSupport.rst (+10-10)
``````````diff
diff --git a/clang/docs/OpenMPSupport.rst b/clang/docs/OpenMPSupport.rst
index 3993911652acc..49bc4d4e8dad8 100644
--- a/clang/docs/OpenMPSupport.rst
+++ b/clang/docs/OpenMPSupport.rst
@@ -268,7 +268,7 @@ implementation.
+------------------------------+--------------------------------------------------------------+--------------------------+-----------------------------------------------------------------------+
| device | use_device_ptr/addr preserve host address
when lookup fails | :good:`done` |
https://github.com/llvm/llvm-project/pull/174659 |
+------------------------------+--------------------------------------------------------------+--------------------------+-----------------------------------------------------------------------+
-| device | iterators in map clause or motion clauses
| :none:`done` |
https://github.com/llvm/llvm-project/pull/159112 |
+| device | iterators in map clause or motion clauses
| :good:`done` |
https://github.com/llvm/llvm-project/pull/159112 |
+------------------------------+--------------------------------------------------------------+--------------------------+-----------------------------------------------------------------------+
| device | indirect clause on declare target directive
| :part:`In Progress` |
|
+------------------------------+--------------------------------------------------------------+--------------------------+-----------------------------------------------------------------------+
@@ -312,7 +312,7 @@ implementation.
+------------------------------+--------------------------------------------------------------+--------------------------+-----------------------------------------------------------------------+
| misc | OMP_NUM_TEAMS and OMP_TEAMS_THREAD_LIMIT env
vars | :good:`done` | D138769
|
+------------------------------+--------------------------------------------------------------+--------------------------+-----------------------------------------------------------------------+
-| misc | 'target_device' selector in context specifier
| :none:`worked on` |
|
+| misc | 'target_device' selector in context specifier
| :part:`worked on` |
|
+------------------------------+--------------------------------------------------------------+--------------------------+-----------------------------------------------------------------------+
| misc | begin/end declare variant
| :good:`done` | D71179
|
+------------------------------+--------------------------------------------------------------+--------------------------+-----------------------------------------------------------------------+
@@ -340,7 +340,7 @@ implementation.
+------------------------------+--------------------------------------------------------------+--------------------------+-----------------------------------------------------------------------+
| OMPT | new 'emi' callbacks for external monitoring
interfaces | :good:`done` |
|
+------------------------------+--------------------------------------------------------------+--------------------------+-----------------------------------------------------------------------+
-| OMPT | device tracing interface
| :none:`in progress` | jplehr
|
+| OMPT | device tracing interface
| :part:`in progress` | jplehr
|
+------------------------------+--------------------------------------------------------------+--------------------------+-----------------------------------------------------------------------+
| task | 'strict' modifier for taskloop construct
| :none:`unclaimed` |
|
+------------------------------+--------------------------------------------------------------+--------------------------+-----------------------------------------------------------------------+
@@ -509,7 +509,7 @@ implementation.
+-------------------------------------------------------------+---------------------------+---------------------------+--------------------------------------------------------------------------+
| pure directives in DO CONCURRENT |
| :none:`unclaimed` |
|
+-------------------------------------------------------------+---------------------------+---------------------------+--------------------------------------------------------------------------+
-| Optional argument for all clauses |
:none:`partial` | :none:`In Progress` | Parse/Sema (nowait):
https://github.com/llvm/llvm-project/pull/159628 |
+| Optional argument for all clauses |
:part:`partial` | :part:`In Progress` | Parse/Sema (nowait):
https://github.com/llvm/llvm-project/pull/159628 |
+-------------------------------------------------------------+---------------------------+---------------------------+--------------------------------------------------------------------------+
| Function references for locator list items |
:none:`unclaimed` | :none:`unclaimed` |
|
+-------------------------------------------------------------+---------------------------+---------------------------+--------------------------------------------------------------------------+
@@ -519,7 +519,7 @@ implementation.
+-------------------------------------------------------------+---------------------------+---------------------------+--------------------------------------------------------------------------+
| Extensions to atomic construct |
:none:`unclaimed` | :none:`unclaimed` |
|
+-------------------------------------------------------------+---------------------------+---------------------------+--------------------------------------------------------------------------+
-| Private reductions | :good:`mostly`
| :none:`unclaimed` |
Parse/Sema:https://github.com/llvm/llvm-project/pull/129938 |
+| Private reductions | :part:`mostly`
| :none:`unclaimed` |
Parse/Sema:https://github.com/llvm/llvm-project/pull/129938 |
| |
| | Codegen:
https://github.com/llvm/llvm-project/pull/134709 |
+-------------------------------------------------------------+---------------------------+---------------------------+--------------------------------------------------------------------------+
| Self maps |
:part:`partial` | :none:`unclaimed` | parsing/sema done:
https://github.com/llvm/llvm-project/pull/129888 |
@@ -584,7 +584,7 @@ implementation.
| need_device_addr modifier for adjust_args clause |
:part:`partial` | :none:`unclaimed` | Parsing/Sema:
https://github.com/llvm/llvm-project/pull/143442 |
| |
| |
https://github.com/llvm/llvm-project/pull/149586 |
+-------------------------------------------------------------+---------------------------+---------------------------+--------------------------------------------------------------------------+
-| need_device_ptr modifier for adjust_args clause |
:part:`unclaimed` | :none:`unclaimed` |
|
+| need_device_ptr modifier for adjust_args clause |
:none:`unclaimed` | :none:`unclaimed` |
|
+-------------------------------------------------------------+---------------------------+---------------------------+--------------------------------------------------------------------------+
| Prescriptive num_threads | :good:`done`
| :none:`unclaimed` |
https://github.com/llvm/llvm-project/pull/160659 |
| |
| |
https://github.com/llvm/llvm-project/pull/146403 |
@@ -659,16 +659,16 @@ considered for standardization. Please post on the
+------------------------------+-----------------------------------------------------------------------------------+--------------------------+--------------------------------------------------------+
|Category | Feature
| Status | Reviews
|
+==============================+===================================================================================+==========================+========================================================+
-| atomic extension | `'atomic' strictly nested within 'teams'
| :good:`prototyped` | D126323
|
+| atomic extension | `'atomic' strictly nested within 'teams'
| :part:`prototyped` | D126323
|
| |
<https://openmp.llvm.org/docs/openacc/OpenMPExtensions.html#atomicWithinTeams>`_
| |
|
+------------------------------+-----------------------------------------------------------------------------------+--------------------------+--------------------------------------------------------+
-| device extension | `'ompx_hold' map type modifier
| :good:`prototyped` | D106509,
D106510 |
+| device extension | `'ompx_hold' map type modifier
| :part:`prototyped` | D106509,
D106510 |
| |
<https://openmp.llvm.org/docs/openacc/OpenMPExtensions.html#ompx-hold>`_
| |
|
+------------------------------+-----------------------------------------------------------------------------------+--------------------------+--------------------------------------------------------+
-| device extension | `'ompx_bare' clause on 'target teams'
construct | :good:`prototyped` |
#66844, #70612 |
+| device extension | `'ompx_bare' clause on 'target teams'
construct | :part:`prototyped` |
#66844, #70612 |
| |
<https://www.osti.gov/servlets/purl/2205717>`_
| |
|
+------------------------------+-----------------------------------------------------------------------------------+--------------------------+--------------------------------------------------------+
-| device extension | Multi-dim 'num_teams' and 'thread_limit'
clause on 'target teams ompx_bare' | :good:`partial` | #99732,
#101407, #102715 |
+| device extension | Multi-dim 'num_teams' and 'thread_limit'
clause on 'target teams ompx_bare' | :part:`partial` | #99732,
#101407, #102715 |
| | construct
| |
|
+------------------------------+-----------------------------------------------------------------------------------+--------------------------+--------------------------------------------------------+
``````````
</details>
https://github.com/llvm/llvm-project/pull/189050
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
