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

Ravi Narayanaswamy via Phabricator via cfe-commits cfe-commits at lists.llvm.org
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.

  rG LLVM Github Monorepo



More information about the cfe-commits mailing list