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

Wed Aug 14 14:40:27 PDT 2019

RaviNarayanaswamy added inline comments.

================
Comment at: clang/docs/OpenMPSupport.rst:222-226
+| device extension             | clause: reverse_offload                                      | :good:`done`         | D52780                                     |
++------------------------------+--------------------------------------------------------------+----------------------+--------------------------------------------+
+| device extension             | clause: atomic_default_mem_order                             | :good:`done`         | D53513                                     |
++------------------------------+--------------------------------------------------------------+----------------------+--------------------------------------------+
+| device extension             | clause: dynamic_allocators                                   | :good:`done`         | D53079                                     |
----------------
ABataev wrote:
> Hahnfeld wrote:
> > Do we have CodeGen for this? I only remember Sema support...
> > 
> > Also, `reverse_offload` is listed again below as `unclaimed`.
> We don't have codegen for this. Same for the reverse_offload and atomic_default_mem_order
As Alexey said codegen is not done so maybe change it to partial.
Regardng reverse_offload, it is debate able whether keep it unclaimed or make it partial.  Yes Sema is done but the hard part for  which nobody has claimed is codegen and libomptarget.  At least for the getting this initial document checked in we should keep it as unclaimed.  Do you have a strong objection against this.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D64375