[Mesa-dev] [PATCH 02/12] nv50: setup scissors on clear_render_target/depth_stencil

[Emil Velikov]

On 21/05/14 19:59, Ilia Mirkin wrote:
> On Wed, May 21, 2014 at 2:57 PM, Emil Velikov <emil.l.velikov at gmail.com> wrote:
>> On 21/05/14 00:39, Ilia Mirkin wrote:
>>> From: Christoph Bumiller <e0425955 at student.tuwien.ac.at>
>>>
>>> [imirkin: add logic to also clear the "regular" scissors]
>> Can you please share why are you've added the "regular" scissors ?
> 
> Because I had to when I was implementing the clear_texture and
> clear_buffer hooks (clear_texture didn't end up making it in quite yet
> because I suck at writing piglit tests, but clear_buffer is there
> now). Basically if scissors are set, they will apply to the clear,
> which is not what we want.
> 
>> Does it matter if we set the screen scissors before the render target params?
> 
> AFAIK, with the exception of a few "go" methods, you can pretty much
> randomize the order in which you set all parameters.
> 

Great, thank you.