Hi Alan, > On Mon, Aug 20, 2007 at 09:25:09AM +0100, Nick Clifton wrote: >> offset &= (bfd_vma)~((1L << s->alignment_power) - 1L); >> >> which I think should satisfy everyone. > > Not me. :-) long might only be 32 bit and bfd_vma 64 bit. > > offset &= ~(((bfd_vma) 1 << s->alignment_power) - 1); Doh! Yes that is better. I will check in this change. Cheers Nick