This is an archive of the discontinued LLVM Phabricator instance.

Add new option -fkeep-persistent-storage-variables to Clang release notes
ClosedPublic

Authored by qianzhen on Jul 17 2023, 11:07 AM.

Details

Summary

This patch updates the Clang release notes with the new option -fkeep-persistent-storage-variables added at https://reviews.llvm.org/D150221.

Diff Detail

Event Timeline

qianzhen created this revision.Jul 17 2023, 11:07 AM
Herald added a project: Restricted Project. · View Herald TranscriptJul 17 2023, 11:07 AM
qianzhen requested review of this revision.Jul 17 2023, 11:07 AM
Herald added a project: Restricted Project. · View Herald TranscriptJul 17 2023, 11:07 AM
Herald added a subscriber: cfe-commits. · View Herald Transcript
hubert.reinterpretcast added inline comments.
clang/docs/ReleaseNotes.rst
278–279

Suggestion: Use em dashes to avoid overuse of commas.

Additionally, I am wondering if the release notes is a reasonable place to add something like: Since this inhibits the merging of the affected variables, the number of individual relocations in the program will generally increase.

qianzhen updated this revision to Diff 541581.Jul 18 2023, 8:52 AM

Update description

This revision is now accepted and ready to land.Jul 20 2023, 9:08 PM