Re: [Intel-gfx] [PATCH] drm/i915: Simplify guard logic for setup_scratch_page()

2018-01-29 Thread Chris Wilson
Quoting Matthew Auld (2018-01-29 10:54:35) > On 29 January 2018 at 10:28, Chris Wilson wrote: > > Older gcc is complaining it can't follow the guards and thinks that > > addr may be used uninitialised > > > > In the process, we can simplify down to one loop, > > add/remove: 0/0 grow/shrink: 0/1 up

Re: [Intel-gfx] [PATCH] drm/i915: Simplify guard logic for setup_scratch_page()

2018-01-29 Thread Matthew Auld
On 29 January 2018 at 10:28, Chris Wilson wrote: > Older gcc is complaining it can't follow the guards and thinks that > addr may be used uninitialised > > In the process, we can simplify down to one loop, > add/remove: 0/0 grow/shrink: 0/1 up/down: 0/-131 (-131) > Function

Re: [Intel-gfx] [PATCH] drm/i915: Simplify guard logic for setup_scratch_page()

2018-01-29 Thread Chris Wilson
Quoting Chris Wilson (2018-01-29 10:28:40) > Older gcc is complaining it can't follow the guards and thinks that > addr may be used uninitialised > > In the process, we can simplify down to one loop, > add/remove: 0/0 grow/shrink: 0/1 up/down: 0/-131 (-131) > Function

[Intel-gfx] [PATCH] drm/i915: Simplify guard logic for setup_scratch_page()

2018-01-29 Thread Chris Wilson
Older gcc is complaining it can't follow the guards and thinks that addr may be used uninitialised In the process, we can simplify down to one loop, add/remove: 0/0 grow/shrink: 0/1 up/down: 0/-131 (-131) Function old new delta setup_scratch_page