[Mesa-dev] [PATCH] r600g: don't change the order of writes in merge_inst_group

[Tilman Sauerbeck]

Vadim Girlin [2011-11-17 03:33]:
> Merge may produce incorrect order of operations for r600-eg:
> 
> 	x: inst1 R0.x, ... ;  //from current group
> 	...
> 	t: inst0 R0.x, ... ;  //from previous group, same destination
> 
> Result of inst1 will be lost.
> 
> So compare destinations and don't allow this.
> 
> Signed-off-by: Vadim Girlin <vadimgirlin at gmail.com>
> ---
> 
> cc: Tilman Sauerbeck <tilman at code-monkey.de>
> 
> This should fix the regression with "r600g: lazy load for AR register" for
> r600/r700.

Confirmed :)

Thanks,
Tilman

-- 
A: Because it messes up the order in which people normally read text.
Q: Why is top-posting such a bad thing?
A: Top-posting.
Q: What is the most annoying thing on usenet and in e-mail?
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/mesa-dev/attachments/20111117/26584079/attachment.pgp>