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

[Johannes Doerfert via Phabricator via cfe-commits]

jdoerfert added inline comments.


================
Comment at: clang/docs/OpenMPSupport.rst:163
++------------------------------+--------------------------------------------------------------+--------------+--------------------------------------------+
+| memory mangagement           | allocate directive and allocate clause                       | claimed      | r355614,r335952                            |
++------------------------------+--------------------------------------------------------------+--------------+--------------------------------------------+
----------------
ABataev wrote:
> Done
Here and above, if there are more revisions, please feel free to add them.


================
Comment at: clang/docs/OpenMPSupport.rst:165
++------------------------------+--------------------------------------------------------------+--------------+--------------------------------------------+
+| OMPD                         | OMPD interfaces                                              | mostly done  |                                            |
++------------------------------+--------------------------------------------------------------+--------------+--------------------------------------------+
----------------
Hahnfeld wrote:
> This is not correct, at least it's not yet upstream.
Is there anything upstreamed? What should I put for status and revisions/reviews?


================
Comment at: clang/docs/OpenMPSupport.rst:205
++------------------------------+--------------------------------------------------------------+--------------+--------------------------------------------+
+| device extension             | clause: device_type                                          | claimed      |                                            |
++------------------------------+--------------------------------------------------------------+--------------+--------------------------------------------+
----------------
ABataev wrote:
> kkwli0 wrote:
> > ABataev wrote:
> > > Can't find this in the standard.
> > Section 2.12.7
> Then it is unclaimed, I think.
The excel sheet did assign some of the claimed tasks to people/companies so I will keep them claimed for now and we should ask if they are actually being worked on or not.


================
Comment at: clang/docs/OpenMPSupport.rst:207
++------------------------------+--------------------------------------------------------------+--------------+--------------------------------------------+
+| device extension             | clause: in_reduction                                         | claimed      |                                            |
++------------------------------+--------------------------------------------------------------+--------------+--------------------------------------------+
----------------
ABataev wrote:
> Done.
I only found a 2017 patch which I added but is this completely done? The category suggest this is device related but I did not find device related code.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D64375