[patch gas]: Add missing parts for pdata/xdata generation
Kai Tietz
ktietz70@googlemail.com
Thu Jul 15 08:54:00 GMT 2010
Hi Dave,
2010/7/15 Dave Korn <dave.korn.cygwin@gmail.com>:
> On 14/07/2010 20:33, Kai Tietz wrote:
>
>> * config/obj-coff-seh.c
>> (seh_getelm_data_size): New.
>> (seh_read_offset): Handle negative values.
>> (obj_coff_seh_push): Handle offset for save-register store.
>> (obj_coff_seh_setframe): Add unwind-information for frame.
>> (seh_store_elm_data): New.
>> (seh_getelm_data_size): Return additionally unaligned element count.
>> (seh_make_unwind_entry): Correct tweak about element count.
>>
>> Tested for x86_64-pc-mingw32. (Build test for i686-pc-cygwin and
>> i686-pc-mingw32). Ok for apply?
>
> There are several repeated constructs like this:
>
>> + *puwop++ = (c->elems[i].offset & 0xff);
>> + *puwop++ = ((c->elems[i].offset >> 8) & 0xff);
>> + *puwop++ = ((c->elems[i].offset >> 16) & 0xff);
>> + *puwop++ = ((c->elems[i].offset >> 24) & 0xff);
>
> Are there not convenient bfd_ endian macros you could use in these cases?
> Please replace with pre-existing macros if any are suitable; otherwise, OK.
Yeah, I can use here bfd_putl32/bfd_putl16 instead. It cleans up code a bit.
So I'll commit it with those changes. Ok?
Cheers,
Kai
--
| (\_/) This is Bunny. Copy and paste
| (='.'=) Bunny into your signature to help
| (")_(") him gain world domination
More information about the Binutils
mailing list