This is an archive of the discontinued LLVM Phabricator instance.

[Clang][doc][NFC]Remove duplicate items in ReleaseNotes
ClosedPublic

Authored by yihanaa on Mar 29 2022, 10:18 AM.

Details

Summary

Remove duplicate items in ReleaseNotes for __builtin_dump_struct, the code changes int patch https://reviews.llvm.org/D122248

Diff Detail

Event Timeline

yihanaa created this revision.Mar 29 2022, 10:18 AM
Herald added a project: Restricted Project. · View Herald TranscriptMar 29 2022, 10:18 AM
yihanaa requested review of this revision.Mar 29 2022, 10:18 AM
Herald added a project: Restricted Project. · View Herald TranscriptMar 29 2022, 10:18 AM
Herald added a subscriber: cfe-commits. · View Herald Transcript
erichkeane accepted this revision.Mar 29 2022, 10:20 AM

2 small bits, otherwise LGTM.

clang/docs/ReleaseNotes.rst
98
117

extranious newline.

This revision is now accepted and ready to land.Mar 29 2022, 10:20 AM
yihanaa updated this revision to Diff 418920.Mar 29 2022, 10:26 AM

Fixed "Description" issue and removed useless newlines

erichkeane accepted this revision.Mar 29 2022, 10:27 AM

LGTM. Let me know if you need this committed for you, just provide "name <email address>" for the commit

yihanaa marked 2 inline comments as done.Mar 29 2022, 10:28 AM

2 small bits, otherwise LGTM.

Thanks for review @erichkeane ,i have update the diff, but i don’t have commit access, can you land this patch for me? Please use “wangyihan 1135831309@qq.com” to commit the change.😉

This revision was landed with ongoing or failed builds.Mar 29 2022, 10:31 AM
This revision was automatically updated to reflect the committed changes.