[PATCH] elf: Use uint64_t for common symbol alignment warning

Collin Funk collin.funk1@gmail.com
Thu Oct 2 03:01:45 GMT 2025


"H.J. Lu" <hjl.tools@gmail.com> writes:

> +		  uint64_t c_align = UINT64_C (1) << common_align;
> +		  uint64_t n_align = UINT64_C (1) << normal_align;
> [...]
> +		      (_("warning: alignment %" PRId64 " of common symbol `%s' in %pB is"
> +			 " greater than the alignment (%" PRId64 ") of its section %pA"),
> +		       c_align, name, common_bfd,
> +		       n_align, h->root.u.def.section);
> [...]
> +		      (_("warning: alignment %" PRId64 " of normal symbol `%s' in %pB"
> +			 " is smaller than %" PRId64 " used by the common definition in %pB"),
> +		       n_align, name, normal_bfd,
> +		       c_align, common_bfd);

Doesn't this cause -Wformat-signedness errors? Since PRId64 is for
int64_t, not uint64_t.

Collin


More information about the Binutils mailing list