This is an archive of the discontinued LLVM Phabricator instance.

Differential D146603

[docs] Document -fomit-frame-pointer
ClosedPublic

Authored by MaskRay on Mar 21 2023, 8:36 PM.

Details

Reviewers
aabhinavg
vitalybuka
Commits
rG5f883cdbfbe2: [docs] Document -fomit-frame-pointer

Diff Detail

Event Timeline

MaskRay created this revision.Mar 21 2023, 8:36 PM
Herald added a project: Restricted Project. · View Herald TranscriptMar 21 2023, 8:36 PM
MaskRay requested review of this revision.Mar 21 2023, 8:36 PM
Herald added a project: Restricted Project. · View Herald TranscriptMar 21 2023, 8:36 PM
Herald added a subscriber: cfe-commits. · View Herald Transcript
MaskRay mentioned this in D145840: [Docs] Added -fomit-frame-pointer and -fno-omit-frame-pointer flag documentation.Mar 21 2023, 8:37 PM
MaskRay updated this revision to Diff 507220.Mar 21 2023, 8:38 PM

.

Harbormaster completed remote builds in B220916: Diff 507220.Mar 21 2023, 9:33 PM
vitalybuka accepted this revision.Mar 22 2023, 9:24 PM
vitalybuka added inline comments.
clang/include/clang/Driver/Options.td
2643

Some stack unwinding cases, such as profilers and sanitizers, may prefer specifying the -fno-omit-frame-pointer option. On many targets, -O1 and higher omit the frame pointer by default.

2645

functions

This revision is now accepted and ready to land.Mar 22 2023, 9:24 PM
MaskRay updated this revision to Diff 507792.Mar 23 2023, 10:16 AM

Adjust some terms.

This revision was landed with ongoing or failed builds.Mar 23 2023, 10:19 AM
Closed by commit rG5f883cdbfbe2: [docs] Document -fomit-frame-pointer (authored by MaskRay). · Explain Why
This revision was automatically updated to reflect the committed changes.
MaskRay marked 2 inline comments as done.
MaskRay added a commit: rG5f883cdbfbe2: [docs] Document -fomit-frame-pointer.

Revision Contents

PathSize
clang/
include/
clang/
Driver/
6 lines

Diff 507220

clang/include/clang/Driver/Options.td

  • This file is larger than 256 KB, so syntax highlighting is disabled by default.
Show First 20 Lines • Show All 2,632 Lines • ▼ Show 20 Linesdef fobjc_disable_direct_methods_for_testing :
HelpText<"Ignore attribute objc_direct so that direct methods can be tested">, HelpText<"Ignore attribute objc_direct so that direct methods can be tested">,
MarshallingInfoFlag<LangOpts<"ObjCDisableDirectMethodsForTesting">>; MarshallingInfoFlag<LangOpts<"ObjCDisableDirectMethodsForTesting">>;
defm objc_avoid_heapify_local_blocks : BoolFOption<"objc-avoid-heapify-local-blocks", defm objc_avoid_heapify_local_blocks : BoolFOption<"objc-avoid-heapify-local-blocks",
CodeGenOpts<"ObjCAvoidHeapifyLocalBlocks">, DefaultFalse, CodeGenOpts<"ObjCAvoidHeapifyLocalBlocks">, DefaultFalse,
PosFlag<SetTrue, [], "Try">, PosFlag<SetTrue, [], "Try">,
NegFlag<SetFalse, [], "Don't try">, NegFlag<SetFalse, [], "Don't try">,
BothFlags<[CC1Option, NoDriverOption], " to avoid heapifying local blocks">>; BothFlags<[CC1Option, NoDriverOption], " to avoid heapifying local blocks">>;
def fomit_frame_pointer : Flag<["-"], "fomit-frame-pointer">, Group<f_Group>; def fomit_frame_pointer : Flag<["-"], "fomit-frame-pointer">, Group<f_Group>,
HelpText<"Omit the frame pointer from functions that don't need it. "
"Some stack unwinding cases like profilers and sanitizers may prefer specifying -fno-omit-frame-pointer. "
vitalybukaUnsubmitted
Done
ReplyInline Actions

Some stack unwinding cases, such as profilers and sanitizers, may prefer specifying the -fno-omit-frame-pointer option. On many targets, -O1 and higher omit the frame pointer by default.

vitalybuka: Some stack unwinding cases, such as profilers and sanitizers, may prefer specifying the -fno…
"On many targets -O1 and higher omit the frame pointer by default. "
"-m[no-]omit-leaf-frame-pointer takes precedence for leaf funcitons">;
vitalybukaUnsubmitted
Done
ReplyInline Actions

functions

vitalybuka: functions
def fopenmp : Flag<["-"], "fopenmp">, Group<f_Group>, Flags<[CC1Option, NoArgumentUnused, FlangOption, FC1Option]>, def fopenmp : Flag<["-"], "fopenmp">, Group<f_Group>, Flags<[CC1Option, NoArgumentUnused, FlangOption, FC1Option]>,
HelpText<"Parse OpenMP pragmas and generate parallel code.">; HelpText<"Parse OpenMP pragmas and generate parallel code.">;
def fno_openmp : Flag<["-"], "fno-openmp">, Group<f_Group>, Flags<[NoArgumentUnused]>; def fno_openmp : Flag<["-"], "fno-openmp">, Group<f_Group>, Flags<[NoArgumentUnused]>;
def fopenmp_version_EQ : Joined<["-"], "fopenmp-version=">, Group<f_Group>, Flags<[CC1Option, NoArgumentUnused]>, def fopenmp_version_EQ : Joined<["-"], "fopenmp-version=">, Group<f_Group>, Flags<[CC1Option, NoArgumentUnused]>,
HelpText<"Set OpenMP version (e.g. 45 for OpenMP 4.5, 50 for OpenMP 5.0). Default value is 50.">; HelpText<"Set OpenMP version (e.g. 45 for OpenMP 4.5, 50 for OpenMP 5.0). Default value is 50.">;
defm openmp_extensions: BoolFOption<"openmp-extensions", defm openmp_extensions: BoolFOption<"openmp-extensions",
LangOpts<"OpenMPExtensions">, DefaultTrue, LangOpts<"OpenMPExtensions">, DefaultTrue,
PosFlag<SetTrue, [CC1Option, NoArgumentUnused], PosFlag<SetTrue, [CC1Option, NoArgumentUnused],
▲ Show 20 Lines • Show All 4,566 Lines • Show Last 20 Lines