[PATCH] D126549: [Clang][OpenMP] Don't overload "extension" in status doc

Joel E. Denny via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri May 27 08:09:34 PDT 2022


jdenny created this revision.
jdenny added reviewers: ABataev, jdoerfert, RaviNarayanaswamy.
jdenny added a project: OpenMP.
Herald added subscribers: guansong, yaxunl.
Herald added a project: All.
jdenny requested review of this revision.
Herald added a subscriber: sstefan1.
Herald added a project: clang.

In Clang's OpenMPSupport.rst, "extension" is currently overloaded to
describe both:

1. Standard OpenMP features that appear only in recent versions of the OpenMP spec.
2. Non-standard features supported by Clang.  This usage appears in the final table on the page.

Last fall, we discussed this issue in the OpenMP in LLVM call and 
agreed it should be corrected.  This patch takes the simple approach
of dropping the word "extension" for all occurrences of the first
usage.  The result seems to read well.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D126549

Files:
  clang/docs/OpenMPSupport.rst

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D126549.432567.patch
Type: text/x-patch
Size: 63809 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20220527/4228bc14/attachment-0001.bin>


More information about the cfe-commits mailing list