[Mesa-dev] [PATCH] i965: Add an INTEL_DEBUG=spill option to test spilling

[Jason Ekstrand]

On Thu, Apr 23, 2015 at 10:12 PM, Jordan Justen
<jordan.l.justen at intel.com> wrote:
> On 2015-04-23 12:28:00, Ilia Mirkin wrote:
>> On Thu, Apr 23, 2015 at 3:24 PM, Jordan Justen
>> <jordan.l.justen at intel.com> wrote:
>> >> > +#define DEBUG_SPILL               (1l << 31)
>> >>
>> >> That seems awkward... did you mean 1U? FWIW mesa's not at all careful
>> >> about that...
>> >
>> > Yeah, I agree. 1l is awkward.
>> >
>> > But I think 1U is just unsigned. I don't think that is guaranteed to
>> > be 64-bit.
>>
>> In fact it's guaranteed to be 32-bit. Which can hold 1 << 31 just fine :)
>
> I think unsigned is only guaranteed to be 16-bits.
> http://en.wikipedia.org/wiki/C_data_types

Guaranteed by the C standard, sure.  However, if you're running the
i965 driver on a 16-bit system, you're going to have bigger problems
than debug options not working. ;-)
--Jason