[PATCH] D64375: [OpenMP][Docs] Provide implementation status details

[Alexey Bataev via Phabricator via cfe-commits]

ABataev added inline comments.


================
Comment at: clang/docs/OpenMPSupport.rst:161
++------------------------------+--------------------------------------------------------------+--------------+--------------------------------------------+
+| memory mangagement           | memory allocators                                            | claimed      |                                            |
++------------------------------+--------------------------------------------------------------+--------------+--------------------------------------------+
----------------
Done


================
Comment at: clang/docs/OpenMPSupport.rst:163
++------------------------------+--------------------------------------------------------------+--------------+--------------------------------------------+
+| memory mangagement           | allocate directive and allocate clause                       | claimed      | r355614,r335952                            |
++------------------------------+--------------------------------------------------------------+--------------+--------------------------------------------+
----------------
Done


================
Comment at: clang/docs/OpenMPSupport.rst:175-177
+| task extension               | clause: depend on the taskwait construct                     | claimed      |                                            |
++------------------------------+--------------------------------------------------------------+--------------+--------------------------------------------+
+| task extension               | depend objects and detachable tasks                          | claimed      |                                            |
----------------
I don't remember anything related to these items. Seems to me, they are unclaimed


================
Comment at: clang/docs/OpenMPSupport.rst:201
++------------------------------+--------------------------------------------------------------+--------------+--------------------------------------------+
+| device extension             | support full 'defaultmap' functionality                      | claimed      |                                            |
++------------------------------+--------------------------------------------------------------+--------------+--------------------------------------------+
----------------
Seems to me it is unclaimed


================
Comment at: clang/docs/OpenMPSupport.rst:205
++------------------------------+--------------------------------------------------------------+--------------+--------------------------------------------+
+| device extension             | clause: device_type                                          | claimed      |                                            |
++------------------------------+--------------------------------------------------------------+--------------+--------------------------------------------+
----------------
Can't find this in the standard.


================
Comment at: clang/docs/OpenMPSupport.rst:207
++------------------------------+--------------------------------------------------------------+--------------+--------------------------------------------+
+| device extension             | clause: in_reduction                                         | claimed      |                                            |
++------------------------------+--------------------------------------------------------------+--------------+--------------------------------------------+
----------------
Done.


================
Comment at: clang/docs/OpenMPSupport.rst:213
++------------------------------+--------------------------------------------------------------+--------------+--------------------------------------------+
+| device extension             | nested target declare                                        | claimed      | D51378                                     |
++------------------------------+--------------------------------------------------------------+--------------+--------------------------------------------+
----------------
Done


================
Comment at: clang/docs/OpenMPSupport.rst:215
++------------------------------+--------------------------------------------------------------+--------------+--------------------------------------------+
+| device extension             | implicitly map 'this' (this[:1])                             | claimed      | D55982                                     |
++------------------------------+--------------------------------------------------------------+--------------+--------------------------------------------+
----------------
Done


================
Comment at: clang/docs/OpenMPSupport.rst:233
++------------------------------+--------------------------------------------------------------+--------------+--------------------------------------------+
+| device extension             | mapping lambda expression                                    | claimed      | D51107                                     |
++------------------------------+--------------------------------------------------------------+--------------+--------------------------------------------+
----------------
Done


================
Comment at: clang/docs/OpenMPSupport.rst:243
++------------------------------+--------------------------------------------------------------+--------------+--------------------------------------------+
+| atomic extension             | hints for the atomic construct                               | done         | D51233                                     |
++------------------------------+--------------------------------------------------------------+--------------+--------------------------------------------+
----------------
This is just the runtime part, the compiler does not support this


================
Comment at: clang/docs/OpenMPSupport.rst:245-247
+| base language                | C11 support                                                  | unclaimed    |                                            |
++------------------------------+--------------------------------------------------------------+--------------+--------------------------------------------+
+| base language                | C++11/14/17 support                                          | unclaimed    |                                            |
----------------
Not sure that there are special requirements for this in the standard


================
Comment at: clang/docs/OpenMPSupport.rst:249
++------------------------------+--------------------------------------------------------------+--------------+--------------------------------------------+
+| base language                | lambda support                                               | claimed      |                                            |
++------------------------------+--------------------------------------------------------------+--------------+--------------------------------------------+
----------------
lambdas are supported.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D64375/new/

https://reviews.llvm.org/D64375