This is an archive of the discontinued LLVM Phabricator instance.

[clang] Document -femit-compact-unwind option in the User’s Manual
ClosedPublic

Authored by int3 on Jul 14 2022, 6:47 AM.

Diff Detail

Event Timeline

int3 created this revision.Jul 14 2022, 6:47 AM
Herald added a project: Restricted Project. · View Herald TranscriptJul 14 2022, 6:47 AM
int3 requested review of this revision.Jul 14 2022, 6:47 AM
Herald added a project: Restricted Project. · View Herald TranscriptJul 14 2022, 6:47 AM
Herald added a subscriber: cfe-commits. · View Herald Transcript
int3 added a comment.Jul 14 2022, 6:48 AM

@davide just adding you in case you have opinions on the wording around the bincompat issue

MaskRay added inline comments.
clang/docs/UsersManual.rst
848

I think flag applies to these boolean options. This is an option, not a flag.

int3 updated this revision to Diff 444740.Jul 14 2022, 11:16 AM
int3 retitled this revision from [clang] Document -femit-compact-unwind flag in the User’s Manual to [clang] Document -femit-compact-unwind option in the User’s Manual.

option

int3 marked an inline comment as done.Jul 14 2022, 11:16 AM
thakis accepted this revision.Jul 14 2022, 12:02 PM
thakis added a subscriber: thakis.

Thanks!

clang/docs/UsersManual.rst
853

nit: I think most things in apple-land call it arm64, not aarch64

This revision is now accepted and ready to land.Jul 14 2022, 12:02 PM
int3 marked an inline comment as done.Jul 14 2022, 1:50 PM
This revision was landed with ongoing or failed builds.Jul 14 2022, 1:50 PM
This revision was automatically updated to reflect the committed changes.