On 10/02/2014 02:00 PM, DJ Delorie wrote:
Ah, good point.  In which case I don't see what this code is trying to
accomplish relative to falling through to the "prefer the unsigned one"
code below.  Shall we just remove it?

I don't know for sure.  There was __int128 code there, I replaced it
with the "same" code, so as to avoid any functional differences on
mainstream targets.

I imagine the code is there for when __int128 is the same size as some
other types besides long long.

But if __int128 happened to be the same size as long the code was wrong. Well, I suppose it could be there to prefer __int128 to intTI_type_node. I guess let's move the intN handling below the code for (u)long and add support for unsigned extended integers like there is for both long and long long.

Jason

Reply via email to