Re: [PATCH v2 06/18] perf test cs-etm: Replace unroll loop thread with deterministic decode test

From: Leo Yan

Date: Wed Jun 03 2026 - 10:18:54 EST


On Tue, Jun 02, 2026 at 03:26:48PM +0100, James Clark wrote:

[...]

> +# Remove open brace lines as they may not be hit depending on the compiler
> +sed -i \
> + -e '/deterministic.c:8$/d' \
> + -e '/deterministic.c:15$/d' \
> + -e '/deterministic.c:23$/d' \
> + "$tmpdir/script"

Is this related to the function definition?

I can see the brace lines with change below. It might be more reliable
if adding unused function argument, which can give chance for hit
function entry.

static int function1(void)
{
...

return 0;
}

Thanks,
Leo