[PATCH] D42352: [ReleaseNotes] Mention OpenMP Tools Interface in runtime library

2018-01-22 Thread Jonas Hahnfeld via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL323179: [ReleaseNotes] Mention OpenMP Tools Interface in runtime library (authored by Hahnfeld, committed by ). Herald added a subscriber: llvm-commits. Changed prior to commit: https://reviews.llvm.org

[PATCH] D42352: [ReleaseNotes] Mention OpenMP Tools Interface in runtime library

2018-01-22 Thread Olga Malysheva via Phabricator via cfe-commits
omalyshe accepted this revision. omalyshe added a comment. LGTM https://reviews.llvm.org/D42352 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D42352: [ReleaseNotes] Mention OpenMP Tools Interface in runtime library

2018-01-22 Thread Jonas Hahnfeld via Phabricator via cfe-commits
Hahnfeld updated this revision to Diff 131004. Hahnfeld marked 3 inline comments as done. Hahnfeld added a comment. Fix `macOS`. https://reviews.llvm.org/D42352 Files: docs/ReleaseNotes.rst Index: docs/ReleaseNotes.rst === ---

[PATCH] D42352: [ReleaseNotes] Mention OpenMP Tools Interface in runtime library

2018-01-22 Thread Olga Malysheva via Phabricator via cfe-commits
omalyshe added inline comments. Comment at: docs/ReleaseNotes.rst:226 +- The LLVM OpenMP runtime `libomp` now supports the OpenMP Tools Interface (OMPT) + on x86, x86_64, AArch64, and PPC64 on Linux, Windows, and mac OS. If you observe + a measurable performance impact on one

[PATCH] D42352: [ReleaseNotes] Mention OpenMP Tools Interface in runtime library

2018-01-22 Thread Jonas Hahnfeld via Phabricator via cfe-commits
Hahnfeld added inline comments. Comment at: docs/ReleaseNotes.rst:226 +- The LLVM OpenMP runtime `libomp` now supports the OpenMP Tools Interface (OMPT) + on x86, x86_64, AArch64, and PPC64 on Linux, Windows, and mac OS. If you observe + a measurable performance impact on one

[PATCH] D42352: [ReleaseNotes] Mention OpenMP Tools Interface in runtime library

2018-01-22 Thread Olga Malysheva via Phabricator via cfe-commits
omalyshe added inline comments. Comment at: docs/ReleaseNotes.rst:226 +- The LLVM OpenMP runtime `libomp` now supports the OpenMP Tools Interface (OMPT) + on x86, x86_64, AArch64, and PPC64 on Linux, Windows, and mac OS. If you observe + a measurable performance impact on one

[PATCH] D42352: [ReleaseNotes] Mention OpenMP Tools Interface in runtime library

2018-01-22 Thread Jonas Hahnfeld via Phabricator via cfe-commits
Hahnfeld added inline comments. Comment at: docs/ReleaseNotes.rst:230 + applications, please rebuild the runtime library with `-DLIBOMP_OMPT_SUPPORT=OFF` + and file a bug at `LLVM's Bugzilla

[PATCH] D42352: [ReleaseNotes] Mention OpenMP Tools Interface in runtime library

2018-01-22 Thread Jonas Hahnfeld via Phabricator via cfe-commits
Hahnfeld updated this revision to Diff 130900. Hahnfeld marked 2 inline comments as done. Hahnfeld added a comment. Address review comments. https://reviews.llvm.org/D42352 Files: docs/ReleaseNotes.rst Index: docs/ReleaseNotes.rst

[PATCH] D42352: [ReleaseNotes] Mention OpenMP Tools Interface in runtime library

2018-01-22 Thread Olga Malysheva via Phabricator via cfe-commits
omalyshe added inline comments. Comment at: docs/ReleaseNotes.rst:227 + on x86, x86_64, AArch64, and PPC64 on Linux, Windows, and mac OS. Performance + measurements show that the changes result in a slowdown of at most 4 percent + in micro-benchmarks. However, if you observe a

[PATCH] D42352: [ReleaseNotes] Mention OpenMP Tools Interface in runtime library

2018-01-22 Thread Hans Wennborg via Phabricator via cfe-commits
hans accepted this revision. hans added a comment. This revision is now accepted and ready to land. Sounds good to me. Comment at: docs/ReleaseNotes.rst:230 + applications, please rebuild the runtime library with `-DLIBOMP_OMPT_SUPPORT=OFF` + and file a bug at `LLVM's Bugzil

[PATCH] D42352: [ReleaseNotes] Mention OpenMP Tools Interface in runtime library

2018-01-21 Thread Jonas Hahnfeld via Phabricator via cfe-commits
Hahnfeld created this revision. Hahnfeld added reviewers: rsmith, ABataev, hans. Herald added subscribers: cfe-commits, guansong. The OpenMP runtime has no dedicated Release Notes, so add it to Clang's section about OpenMP. Repository: rC Clang https://reviews.llvm.org/D42352 Files: docs/