On 05/11/2015 02:06 PM, Chen Gang wrote: > On 5/12/15 00:01, Richard Henderson wrote: >> On 05/10/2015 03:42 PM, Chen Gang wrote: >>> -static __inline unsigned int >>> +static inline uint8_t >>> get_BFEnd_X0(tilegx_bundle_bits num) >> >> Do not change these casts to uint8_t. It's unnecessary churn. >> > > For me, it is enough to return uint8_t, and the caller really treats it > as uint8_t. So for the function declaration, uint8_t is more precise > than unsigned int for return type.
I don't want to argue about this anymore. Drop all the uint8_t and uint16_t. r~