This is an archive of the discontinued LLVM Phabricator instance.

[LLD] [docs] Amend the release notes with more things I've contributed since the last release
ClosedPublic

Authored by mstorsjo on Jan 21 2019, 1:42 AM.

Details

Summary

I earlier wrote the most of the relnotes for things I've contributed this cycle, but I found a few more notable things while looking through the changelog.

Diff Detail

Repository
rL LLVM

Event Timeline

mstorsjo created this revision.Jan 21 2019, 1:42 AM
hans accepted this revision.Jan 22 2019, 10:37 AM

lgtm

This revision is now accepted and ready to land.Jan 22 2019, 10:37 AM
ruiu added inline comments.Jan 22 2019, 10:39 AM
docs/ReleaseNotes.rst
73 ↗(On Diff #182755)

I don't know if I get the meaning of this sentence correctly. The option is accepted and ignored, /therefore/ no build id was generated?

mstorsjo marked an inline comment as done.Jan 22 2019, 10:45 AM
mstorsjo added inline comments.
docs/ReleaseNotes.rst
73 ↗(On Diff #182755)

Well, we still ignore it in the mingw frontend and generate the buildid in COFF/Writer.cpp regardless if the flag was set.

ruiu added inline comments.Jan 22 2019, 10:53 AM
docs/ReleaseNotes.rst
73 ↗(On Diff #182755)

How about something like this? Previously, the build-id option did not actually generate a build id unless --pdb was specified.

mstorsjo marked an inline comment as done.Jan 22 2019, 10:59 AM
mstorsjo added inline comments.
docs/ReleaseNotes.rst
73 ↗(On Diff #182755)

That sounds good, thanks!

ruiu accepted this revision.Jan 22 2019, 11:04 AM

LGTM

I can submit this for you if you don't know how to do this to the 8.0 branch.

LGTM

I can submit this for you if you don't know how to do this to the 8.0 branch.

Thanks, but I think I know how to get it committed on the branch!

This revision was automatically updated to reflect the committed changes.