[PATCH] D11059: [OPENMP] Info in release notes about OpenMP support in clang.

Hans Wennborg hans at chromium.org
Wed Jul 22 19:01:49 PDT 2015


hans added a comment.

In http://reviews.llvm.org/D11059#210394, @fraggamuffin wrote:

> +Jopnathan Peyton
>
> We had a discussion on this on our Wednesday morning call. Jonathan Peyton
>  has added the cmake infrastructure to support this and is automating the
>  library tests to support the switch for libomp. Jonathan will be able to
>  correct me if I am wrong. This we hope will enable the setting of the
>  default switch of -fopenmp. If there are additional requirements, please
>  let us know. Thanks.


I would like to see that setting enabled first, before claiming full support in the release notes, though. If the release notes say it's fully supported, it needs to Just Work out of the box.

Also, I do want to point out that it's now pretty late for this kind of changes in the 3.7 process. Not necessarily too late, but pretty late. It would be good to have a plan for the potential scenario where the default switch doesn't make it into this release: even if it's not on by default, how do we get the most value out of it in this release, can it be made easy for users to experiment with it and provide feedback, etc.


http://reviews.llvm.org/D11059







More information about the cfe-commits mailing list