On Wed, Aug 29, 2012 at 10:32 AM, Florian Weimer <f...@deneb.enyo.de> wrote:
>
> This patches fixes an integer overflow in libiberty, which leads to
> crashes in binutils.  The long version of the objalloc_alloc macro
> would have needed another conditional, so I removed that and replaced
> it with a call to the actual implementation.

I guess I don't see why removing the macro is desirable.  In many uses
of objalloc_alloc the conditional can be optimized out anyhow.  It's
true that it can't always be, but, so what?  The macro is probably
still a win.

Or perhaps the macro is not a win, but that would be a decision to
make separately from adding this conditional.

Ian

Reply via email to