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



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

Reply via email to