On 10/02/2014 12:41 PM, DJ Delorie wrote:
The C++ standard says that extended integer types participate in the
usual arithmetic conversions. If I add a 32-bit int and an __int48, the
usual arithmetic conversions should convert the int to __int48.
Except the code you're referring to isn't part of that conversion. It
only handles types that are the same size, not different sizes
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?
The test would only pass for msp430x (and fail for msp430, which is
the same target back-end). Do I need to redo the big patch, or would
a separate one suffice?
Separate is fine.
Jason
