This is an archive of the discontinued LLVM Phabricator instance.

[Clang][OpenMP] Don't overload "extension" in status doc
ClosedPublic

Authored by jdenny on May 27 2022, 8:09 AM.

Details

Summary

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.

Diff Detail

Event Timeline

jdenny created this revision.May 27 2022, 8:09 AM
Herald added a project: Restricted Project. · View Herald TranscriptMay 27 2022, 8:09 AM
jdenny requested review of this revision.May 27 2022, 8:09 AM
Herald added a project: Restricted Project. · View Herald TranscriptMay 27 2022, 8:09 AM
Herald added a subscriber: sstefan1. · View Herald Transcript
This revision is now accepted and ready to land.Jun 27 2022, 1:07 PM
This revision was automatically updated to reflect the committed changes.