This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [tree-ssa][rfc] -finstrument-functions at tree level



On Dec 10, 2003, at 17:50, Richard Henderson wrote:


Even after looking at this and preserving apparently intended
semantics, I don't see that those semantics are useful.  Given

	void a() { }
	void noinline b() { a(); }
	void c() { b(); }

is __cyg_profile_func_start(a, c) really meaningful?  I'm not
sure that it is.  I would sorta expect that inlined functions
don't get noticed by this hook at all.

Thoughts?


+ /* If we're instrumenting function entry/exit, then prepend the call to
+ the entry hook and wrap the whole function in a TRY_FINALLY_EXPR to
+ catch the exit hook. */
+ /* ??? Add some way to ignore exceptions for this TFE. */

There was a PR about getting -finstrument-functions to work with exceptions, see PR 2200.
I would just remove the ??? because of that PR or add another flag to control it.


Thanks,
Andrew Pinski


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]