ABataev added inline comments.

================
Comment at: clang/docs/OpenMPSupport.rst:194
 
+------------------------------+--------------------------------------------------------------+--------------------------+-----------------------------------------------------------------------+
-| device extension             | requires directive (unified shared memory)    
               | :good:`done`             |                                     
                                  |
+| device extension             | requires directive                            
               | :good:`done`             |                                     
                                  |
 
+------------------------------+--------------------------------------------------------------+--------------------------+-----------------------------------------------------------------------+
----------------
We have support only for unified memory, so must be `partial`


================
Comment at: clang/docs/OpenMPSupport.rst:210
 
+------------------------------+--------------------------------------------------------------+--------------------------+-----------------------------------------------------------------------+
-| device extension             | map(replicate) or map(local) when requires 
unified_shared_me | :part:`worked on`        | D55719,D55892                    
                                     |
+| device extension             | map(local) when requires 
unified_shared_memory               | :part:`worked on`        | D55719,D55892  
                                                       |
 
+------------------------------+--------------------------------------------------------------+--------------------------+-----------------------------------------------------------------------+
----------------
I assume, it is `done`.


================
Comment at: clang/docs/OpenMPSupport.rst:216
 
+------------------------------+--------------------------------------------------------------+--------------------------+-----------------------------------------------------------------------+
+| device extension             | pointer attachment                            
               | :none:`unclaimed`        |                                     
                                  |
++------------------------------+--------------------------------------------------------------+--------------------------+-----------------------------------------------------------------------+
----------------
Is this for Fortran?


================
Comment at: clang/docs/OpenMPSupport.rst:240
++------------------------------+--------------------------------------------------------------+--------------------------+-----------------------------------------------------------------------+
+| misc extension               | prevent new type definitions in clauses       
               | :none:`unclaimed`        |                                     
                                  |
 
+------------------------------+--------------------------------------------------------------+--------------------------+-----------------------------------------------------------------------+
----------------
What is this?


================
Comment at: clang/docs/OpenMPSupport.rst:242
 
+------------------------------+--------------------------------------------------------------+--------------------------+-----------------------------------------------------------------------+
-| misc extensions              | prevent new type definitions in clauses       
               | :none:`unclaimed`        |                                     
                                  |
+| memory model extension       | memory model update                           
               | :none:`unclaimed`        |                                     
                                  |
 
+------------------------------+--------------------------------------------------------------+--------------------------+-----------------------------------------------------------------------+
----------------
What kind of memory model update?


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

https://reviews.llvm.org/D72901



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

Reply via email to