[PATCH] D40276: Add -finstrument-function-entry-bare flag

Egor Pasko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Nov 21 04:43:07 PST 2017


pasko accepted this revision.
pasko added a comment.
This revision is now accepted and ready to land.

Instrumenting the function entry post-inlining, without function exit, and with no parameters is exactly what we need. The `__cyg_profile_func_enter_bare` sounds good to me as a name. Thank you!

Unnecessary thoughts just to get a feeling we are on the same page: this could theoretically be made more orthogonal where `-finstrument-functions-after-inlining` could regulate whether the call is pre- or post-inlining, but I don't see how pre-inlining without parameters would be usable without too much DWARF digging, which is not too practical.


https://reviews.llvm.org/D40276





More information about the cfe-commits mailing list