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

Mon Jul 8 16:09:33 PDT 2019

ABataev added a comment.

The scheme itself looks good in general. Just as an improvement, it would be good to use coloring for the feature status.
Also, what's the difference between claimed, mostly done and done?

================
Comment at: clang/docs/OpenMPSupport.rst:153
++------------------------------+--------------------------------------------------------------+--------------+--------------------------------------------+
+| loop extension               | collapse non-rectangular nested loop                         | unclaimed    |                                            |
++------------------------------+--------------------------------------------------------------+--------------+--------------------------------------------+
----------------
I'm working on this feature, but don't know when it's going to be ready.

================
Comment at: clang/docs/OpenMPSupport.rst:219
++------------------------------+--------------------------------------------------------------+--------------+--------------------------------------------+
+| device extension             | Fortran pointer mapping                                      | unclaimed    |                                            |
++------------------------------+--------------------------------------------------------------+--------------+--------------------------------------------+
----------------
Do we really need Fortran stuff here?

================
Comment at: clang/docs/OpenMPSupport.rst:237
++------------------------------+--------------------------------------------------------------+--------------+--------------------------------------------+
+| device extension             | map(replicate) or map(local) when requires unified_shared_me | done         | D55719,D55892                              |
++------------------------------+--------------------------------------------------------------+--------------+--------------------------------------------+
----------------
Not sure 100%, but seems to me it is not done.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D64375