[PATCH] R600/SI: SI Control Flow Annotation bug fixed

Christian König christian.koenig at amd.com
Tue Jun 3 02:16:13 PDT 2014


Am 03.06.2014 10:59, schrieb Matt Arsenault:
> On Jun 3, 2014, at 1:46 AM, Елена Денисова <denisova.lena at gmail.com> wrote:
>
>> <0001-SI-Control-Flow-Annotation-bug-fix.patch>
> Needs a test

The control flow leading to this issue is rather complicated, and 
because of this I'm not sure if we want to have a test case for it cause 
it would break quite often on completely unrelated changes.

On the other hand I know that Lena doesn't have an environment to run 
piglit tests with this, so somebody (Tim, Matt?) should do this to make 
sure we don't break anything there.

Either way the patch itself looks good to me and is Reviewed-by: 
Christian König <christian.koenig at amd.com>

Regards,
Christian.




More information about the llvm-commits mailing list