Why is this one needed? That's what the ATTRIBUTE_UNUSED is for...
The rest of it looks OK to me. I assume you're done the usual
build/test cycle, yes?
> @@ -931,6 +947,8 @@
> struct bfd_hash_entry *h;
> PTR string ATTRIBUTE_UNUSED;
> {
> + (void)string;
> +
> if (pe_dll_extra_pe_debug)