Hahnfeld added inline comments.

================
Comment at: clang/docs/OpenMPSupport.rst:233
++------------------------------+--------------------------------------------------------------+--------------+--------------------------------------------+
+| device extension             | mapping lambda expression                     
               | claimed      | D51107                                     |
++------------------------------+--------------------------------------------------------------+--------------+--------------------------------------------+
----------------
ABataev wrote:
> kkwli0 wrote:
> > ABataev wrote:
> > > Done
> > Do we support the behavior in 318:7-14?
> Yes.
ping @jdoerfert 


================
Comment at: clang/docs/OpenMPSupport.rst:250
++------------------------------+--------------------------------------------------------------+----------------------+--------------------------------------------+
+| misc extension               | library shutdown (omp_pause_resource[_all])   
               | :part:`worked on`    |                                         
   |
++------------------------------+--------------------------------------------------------------+----------------------+--------------------------------------------+
----------------
Hahnfeld wrote:
> I think this should say partial, it's present in `libomp` after D55078.
ping, I still think `partial` is more correct


================
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                                  
   |
----------------
Do we have CodeGen for this? I only remember Sema support...

Also, `reverse_offload` is listed again below as `unclaimed`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D64375



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to