Re: [Mesa-dev] [PATCH 1/3] R600: Expand v2i32 load/store instead of custom lowering

2013-06-20 Thread Aaron Watry
Thanks Tom. Also as discussed on IRC, I checked the v2i32 expand change against piglits cl/program/execute/*stack*.cl test cases, as well as the v2i32 test that just sent to the piglit list. The tests were run on a Radeon 5400, and all of them passed with the Expand'ed v2i32 loads/stores. --Aaron

Re: [Mesa-dev] [PATCH 1/3] R600: Expand v2i32 load/store instead of custom lowering

2013-06-20 Thread Tom Stellard
On Wed, Jun 19, 2013 at 08:51:01AM -0700, Tom Stellard wrote: > On Mon, Jun 17, 2013 at 04:11:38PM -0500, Aaron Watry wrote: > > The custom lowering causes llc to crash with a segfault. > > > > Ideally, the custom lowering can be fixed, but this allows > > programs which load/store v2i32 to work w

Re: [Mesa-dev] [PATCH 1/3] R600: Expand v2i32 load/store instead of custom lowering

2013-06-19 Thread Tom Stellard
On Mon, Jun 17, 2013 at 04:11:38PM -0500, Aaron Watry wrote: > The custom lowering causes llc to crash with a segfault. > > Ideally, the custom lowering can be fixed, but this allows > programs which load/store v2i32 to work without crashing. > > Signed-off-by: Aaron Watry > --- > lib/Target/R60

[Mesa-dev] [PATCH 1/3] R600: Expand v2i32 load/store instead of custom lowering

2013-06-17 Thread Aaron Watry
The custom lowering causes llc to crash with a segfault. Ideally, the custom lowering can be fixed, but this allows programs which load/store v2i32 to work without crashing. Signed-off-by: Aaron Watry --- lib/Target/R600/R600ISelLowering.cpp | 4 ++-- test/CodeGen/R600/load.vec.ll| 6 +++