[PATCH] D72901: [OpenMP] [DOCS] Update OMP5.0 feature status table [NFC]

This revision was automatically updated to reflect the committed changes. Closed by commit rGac430336318a: [OpenMP] [DOCS] Update OMP5.0 feature status table [NFC] (authored by kkwli0). Herald added a project: clang. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D72901: [OpenMP] [DOCS] Update OMP5.0 feature status table [NFC]

ABataev accepted this revision. ABataev added a comment. This revision is now accepted and ready to land. LG, but still would be good to add some references to the standard. Some of the features use special wording, which very hard to find in the standard. CHANGES SINCE LAST ACTION

[PATCH] D72901: [OpenMP] [DOCS] Update OMP5.0 feature status table [NFC]

kkwli0 marked 2 inline comments as done. kkwli0 added inline comments. Comment at: clang/docs/OpenMPSupport.rst:216

[PATCH] D72901: [OpenMP] [DOCS] Update OMP5.0 feature status table [NFC]

kkwli0 updated this revision to Diff 242166. kkwli0 added a comment. Change `requires unified_address` status to partial. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D72901/new/ https://reviews.llvm.org/D72901 Files: clang/docs/OpenMPSupport.rst Index:

[PATCH] D72901: [OpenMP] [DOCS] Update OMP5.0 feature status table [NFC]

ABataev added inline comments. Comment at: clang/docs/OpenMPSupport.rst:194 +--+--+--+---+ -|

[PATCH] D72901: [OpenMP] [DOCS] Update OMP5.0 feature status table [NFC]

kkwli0 added a comment. Ping 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

[PATCH] D72901: [OpenMP] [DOCS] Update OMP5.0 feature status table [NFC]

kkwli0 updated this revision to Diff 239719. kkwli0 marked 2 inline comments as done. kkwli0 added a comment. Herald added a subscriber: jfb. Address review comments and rebase. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D72901/new/ https://reviews.llvm.org/D72901 Files:

[PATCH] D72901: [OpenMP] [DOCS] Update OMP5.0 feature status table [NFC]

kkwli0 marked 13 inline comments as done. kkwli0 added inline comments. Comment at: clang/docs/OpenMPSupport.rst:216

[PATCH] D72901: [OpenMP] [DOCS] Update OMP5.0 feature status table [NFC]

dreachem added inline comments. Comment at: clang/docs/OpenMPSupport.rst:210 +--+--+--+---+ -|

[PATCH] D72901: [OpenMP] [DOCS] Update OMP5.0 feature status table [NFC]

jdoerfert added a comment. @kkwli0 I propose you can merge the parts where discussion has reached a consensus while the other parts are resolved. I'm generally fine with this, we can always improve on it further. Comment at: clang/docs/OpenMPSupport.rst:194

[PATCH] D72901: [OpenMP] [DOCS] Update OMP5.0 feature status table [NFC]

ABataev added inline comments. Comment at: clang/docs/OpenMPSupport.rst:242 +--+--+--+---+ -|

[PATCH] D72901: [OpenMP] [DOCS] Update OMP5.0 feature status table [NFC]

kkwli0 marked 2 inline comments as done. kkwli0 added inline comments. Comment at: clang/docs/OpenMPSupport.rst:242

[PATCH] D72901: [OpenMP] [DOCS] Update OMP5.0 feature status table [NFC]

kkwli0 marked an inline comment as done. kkwli0 added inline comments. Comment at: clang/docs/OpenMPSupport.rst:194

[PATCH] D72901: [OpenMP] [DOCS] Update OMP5.0 feature status table [NFC]

ABataev added inline comments. Comment at: clang/docs/OpenMPSupport.rst:240 ++--+--+--+---+ +| misc

[PATCH] D72901: [OpenMP] [DOCS] Update OMP5.0 feature status table [NFC]

kkwli0 marked 2 inline comments as done. kkwli0 added inline comments. Comment at: clang/docs/OpenMPSupport.rst:240

[PATCH] D72901: [OpenMP] [DOCS] Update OMP5.0 feature status table [NFC]

jdoerfert added inline comments. Comment at: clang/docs/OpenMPSupport.rst:194 +--+--+--+---+ -|

[PATCH] D72901: [OpenMP] [DOCS] Update OMP5.0 feature status table [NFC]

ABataev added inline comments. Comment at: clang/docs/OpenMPSupport.rst:194 +--+--+--+---+ -|

[PATCH] D72901: [OpenMP] [DOCS] Update OMP5.0 feature status table [NFC]

kkwli0 created this revision. kkwli0 added reviewers: ABataev, RaviNarayanaswamy, jdoerfert. Herald added a subscriber: guansong. Add missing OMP5.0 features. https://reviews.llvm.org/D72901 Files: clang/docs/OpenMPSupport.rst Index: clang/docs/OpenMPSupport.rst