Re: [PATCH] intprops: add WRAPV and const flavors for GCC 5

[Paul Eggert]

Pádraig Brady wrote:
> The issue I have with the example is that INT_MULTIPLY_WRAPV()
> takes 3 arguments, while only 2 are provided.

Ouch, right! I didn't see that. Fixed with the attached.

> In case gcc decides to continue to warn in this case, I suppose we could avoid
> the undefined code by promoting to a wider type and explicitly truncating
> the result (though that might need __int128 if available for your new 64 bit 
> case).

I didn't want to go that route, due to the portability hassles of dealing with 
__int128 support. (Besides, if we wanted to deal with __int128, we'd want to 
deal with it as an operand type too, which means we'd need __int256.... :-)

0001-intprops-comment-fix.patch
Description: Text Data