The OpenMP runtime has no dedicated Release Notes, so add it to Clang's section about OpenMP.
Details
Details
Diff Detail
Diff Detail
- Repository
- rL LLVM
Event Timeline
Comment Actions
Sounds good to me.
docs/ReleaseNotes.rst | ||
---|---|---|
230 ↗ | (On Diff #130806) | Is there a missing right-angle bracket, >, here? |
docs/ReleaseNotes.rst | ||
---|---|---|
227 ↗ | (On Diff #130806) | I'd suggest not mentioning any percentage. A note about possible performance issue due to enabling OMPT would be enough. |
Comment Actions
Address review comments.
docs/ReleaseNotes.rst | ||
---|---|---|
230 ↗ | (On Diff #130806) | Yes, and an underscore. I though I was smart enough to write reStructuredText without actually "testing" it... |
docs/ReleaseNotes.rst | ||
---|---|---|
226 ↗ | (On Diff #130900) | Linux*, Windows* and macOS* |
docs/ReleaseNotes.rst | ||
---|---|---|
226 ↗ | (On Diff #130900) | That's done nowhere else in Clang's documentation so I'm strongly opposed to doing it here unless you have a very good legal reason that nobody else knows about. |
docs/ReleaseNotes.rst | ||
---|---|---|
226 ↗ | (On Diff #130900) | I'm OK with omitting "*" but "macOS" should be definitely fixed ;) |