This is an archive of the discontinued LLVM Phabricator instance.

[OpenMP]Update to clang 3.8 ReleaseNotes.rst
ClosedPublic

Authored by fraggamuffin on Feb 16 2016, 7:06 PM.

Details

Summary

This updates the clang 3.8 release notes with OpenMP support statement.

Diff Detail

Event Timeline

fraggamuffin retitled this revision from to [OpenMP]Update to clang 3.8 ReleaseNotes.rst.
fraggamuffin updated this object.

Fix spelling in dependencies

ABataev edited edge metadata.EditedFeb 16 2016, 8:46 PM

Michael, we have some updates also:

Several new features are supported by Clang:

- Array sections [2.4, Array Sections].

- Directive name modifiers for ``if`` clause [2.12, if Clause].

- ``linear`` clause can be used in loop-based directives [2.7.2, loop Construct].

- ``simdlen`` clause [2.8, SIMD Construct].

- ``hint`` clause [2.13.2, critical Construct].

- Parsing/semantic analysis of all non-offloading directives introduced in OpenMP 4.5.

The codegen for OpenMP constructs was significantly improved allowing to produce much more stable and fast code.

Could you add it to your patch?

kkwli0 edited edge metadata.Feb 16 2016, 8:50 PM

OpenMP spec uses the term "device constructs". How about using "non-device directives" instead of "non-offloading directives"?

Along with Alexey's addition, this looks good.

(Don't treat my approval as "LGTM", though -- as I don't have authority for this.)

Yours,
Andrey

fraggamuffin edited edge metadata.

Should we mention that the runtime can be built "in-tree" as well? This simplifies users experience to access OpenMP support.

I did a test. I found the library is built as expected. Without installation process, it can not be used directly. People can then set environment vars to use the header file and the library.

I am not sure how each Linux distribution handle this. For example, when you do apt-get install clang, will the OpenMP runtime be part of it?

Ok, just adding that the runtime can be built in-tree. I will upstream tonight at 7 pm ET.

hans accepted this revision.Feb 23 2016, 1:41 PM
hans edited edge metadata.

Ok, just adding that the runtime can be built in-tree. I will upstream tonight at 7 pm ET.

Looks like this was committed in r261526. Thanks!

This revision is now accepted and ready to land.Feb 23 2016, 1:41 PM
hans closed this revision.Feb 23 2016, 1:41 PM