[prev in list] [next in list] [prev in thread] [next in thread] 

List:       cfe-commits
Subject:    Re: [PATCH] D11059: [OPENMP] Info in release notes about OpenMP support in clang.
From:       Richard Smith <richard () metafoo ! co ! uk>
Date:       2015-07-21 20:24:03
Message-ID: 490e2c1714f578a1e4b9402c5b1ee6b9 () localhost ! localdomain
[Download RAW message or body]

rsmith accepted this revision.
rsmith added a comment.
This revision is now accepted and ready to land.

This looks fine to me. Please remember that this should be committed to the 3.7 \
branch, not to Clang trunk.

Regarding Michael Wong's suggestion:

- The addition of "We plan to continue work on 4.0 for clang 3.8. Please see this \
link for up-to-date status \
https://github.com/clang-omp/clang/wiki/Status-of-supported-OpenMP-constructs_." is \
useful information, but doesn't seem directly relevant for the release notes for \
Clang 3.7, because it's not information about the 3.7 release. I'm happy with this \
                with or without that change.
- We have historically not included lists of contributors in our release notes. While \
attribution of this kind is important, a better place for it is probably LLVM's \
CREDITS.TXT or similar rather than here (we want these credits to remain for the \
lifetime of the project, not just for one release).


http://reviews.llvm.org/D11059




_______________________________________________
cfe-commits mailing list
cfe-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits


[prev in list] [next in list] [prev in thread] [next in thread]