Re: [PATCH 1/1] objtool/klp: Fix is_uncorrelated_static_local() for Clang naming
From: Song Liu
Date: Fri Apr 10 2026 - 18:24:05 EST
On Wed, Apr 8, 2026 at 7:49 AM Joe Lawrence <joe.lawrence@xxxxxxxxxx> wrote:
>
> For naming function-local static locals, GCC uses "<var>.<id>":
> e.g. __already_done.15
>
> while Clang uses "<func>.<var>" with optional ".<id>"
> e.g. create_worker.__already_done.111
>
> The existing is_uncorrelated_static_local() check only matches the GCC
> convention where the variable name is a prefix. Handle both cases by
> checking for a prefix match (GCC) and by checking after the first dot
> separator (Clang).
>
> Fixes: dd590d4d57eb ("objtool/klp: Introduce klp diff subcommand for diffing object files")
> Signed-off-by: Joe Lawrence <joe.lawrence@xxxxxxxxxx>
LGTM.
Acked-by: Song Liu <song@xxxxxxxxxx>