https://bugs.freedesktop.org/show_bug.cgi?id=65873
Aaron Watry changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://bugs.freedesktop.org/show_bug.cgi?id=65873
--- Comment #14 from Aaron Watry ---
I still get errors with the v8i32/v16i32 loads, but those aren't currently
enabled in the R600 or SI back-ends as far as I know.
If I comment out the vload8 and vload16 tests everything runs correctly for me
https://bugs.freedesktop.org/show_bug.cgi?id=65873
--- Comment #13 from Tom Stellard ---
(In reply to comment #9)
> The piglit test should work with this patch:
> http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20130624/179364.
> html
>
> But we should still try to fix the bug in lib
https://bugs.freedesktop.org/show_bug.cgi?id=65873
Aaron Watry changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://bugs.freedesktop.org/show_bug.cgi?id=65873
--- Comment #14 from Aaron Watry ---
I still get errors with the v8i32/v16i32 loads, but those aren't currently
enabled in the R600 or SI back-ends as far as I know.
If I comment out the vload8 and vload16 tests everything runs correctly for me
https://bugs.freedesktop.org/show_bug.cgi?id=65873
--- Comment #13 from Tom Stellard ---
(In reply to comment #9)
> The piglit test should work with this patch:
> http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20130624/179364.
> html
>
> But we should still try to fix the bug in lib
https://bugs.freedesktop.org/show_bug.cgi?id=65873
--- Comment #12 from Aaron Watry ---
No problem. I'll just revert the vload/vstore code to pure CLC implementations
and then override with assembly in the R600 back-end.
--
You are receiving this mail because:
You are the assignee for the bug.
https://bugs.freedesktop.org/show_bug.cgi?id=65873
--- Comment #12 from Aaron Watry ---
No problem. I'll just revert the vload/vstore code to pure CLC implementations
and then override with assembly in the R600 back-end.
--
You are receiving this mail because:
You are the assignee for the bug.
https://bugs.freedesktop.org/show_bug.cgi?id=65873
--- Comment #11 from Tom Stellard ---
(In reply to comment #10)
> I've got a draft fix in my libclc repo on fdo.o. I've tested for
> regressions on Cedar (successfully), and I will test on SI with your 64-bit
> load patch asap (hopefully tonight
https://bugs.freedesktop.org/show_bug.cgi?id=65873
--- Comment #10 from Aaron Watry ---
I've got a draft fix in my libclc repo on fdo.o. I've tested for regressions
on Cedar (successfully), and I will test on SI with your 64-bit load patch asap
(hopefully tonight, but we'll see).
--
You are re
https://bugs.freedesktop.org/show_bug.cgi?id=65873
--- Comment #11 from Tom Stellard ---
(In reply to comment #10)
> I've got a draft fix in my libclc repo on fdo.o. I've tested for
> regressions on Cedar (successfully), and I will test on SI with your 64-bit
> load patch asap (hopefully tonight
https://bugs.freedesktop.org/show_bug.cgi?id=65873
--- Comment #9 from Tom Stellard ---
The piglit test should work with this patch:
http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20130624/179364.html
But we should still try to fix the bug in libclc.
--
You are receiving this mail
https://bugs.freedesktop.org/show_bug.cgi?id=65873
--- Comment #8 from Aaron Watry ---
"Pointers on SI are 64-bit."
Talk about a "duh" moment. I'll see what I can do to fix and test the
vload/vstore implementation pointer generation since this is broken on SI,
maybe 64-bit PTX, and would also be
https://bugs.freedesktop.org/show_bug.cgi?id=65873
--- Comment #7 from Tom Stellard ---
The SI backend needs to be fixed so it can handle zext i64 loads, but there is
also a bug in libclc. The vload implementation assumes pointer types are
32-bit:
define <2 x i32> @__clc_vload2_impl_i32__global
https://bugs.freedesktop.org/show_bug.cgi?id=65873
--- Comment #10 from Aaron Watry ---
I've got a draft fix in my libclc repo on fdo.o. I've tested for regressions
on Cedar (successfully), and I will test on SI with your 64-bit load patch asap
(hopefully tonight, but we'll see).
--
You are re
https://bugs.freedesktop.org/show_bug.cgi?id=65873
--- Comment #9 from Tom Stellard ---
The piglit test should work with this patch:
http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20130624/179364.html
But we should still try to fix the bug in libclc.
--
You are receiving this mail
https://bugs.freedesktop.org/show_bug.cgi?id=65873
--- Comment #8 from Aaron Watry ---
"Pointers on SI are 64-bit."
Talk about a "duh" moment. I'll see what I can do to fix and test the
vload/vstore implementation pointer generation since this is broken on SI,
maybe 64-bit PTX, and would also be
https://bugs.freedesktop.org/show_bug.cgi?id=65873
--- Comment #7 from Tom Stellard ---
The SI backend needs to be fixed so it can handle zext i64 loads, but there is
also a bug in libclc. The vload implementation assumes pointer types are
32-bit:
define <2 x i32> @__clc_vload2_impl_i32__global
https://bugs.freedesktop.org/show_bug.cgi?id=65873
Aaron Watry changed:
What|Removed |Added
Attachment #80972|0 |1
is obsolete|
https://bugs.freedesktop.org/show_bug.cgi?id=65873
Aaron Watry changed:
What|Removed |Added
Attachment #80971|0 |1
is obsolete|
https://bugs.freedesktop.org/show_bug.cgi?id=65873
--- Comment #4 from Aaron Watry ---
The good news is that the store error is gone...
Now I get a load error :/
LLVM ERROR: Cannot select: 0x2c92af0: i64,ch = load 0x21fe0c8, 0x2c924f0,
0x2c921f0 [ID=15]
0x2c924f0: i64 = add 0x2c91ef0, 0x2c923
https://bugs.freedesktop.org/show_bug.cgi?id=65873
--- Comment #3 from Tom Stellard ---
These patches should fix the issue:
http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20130617/178156.html
The first patch is unrelated to the bug, but it is required in order to prevent
the test c
https://bugs.freedesktop.org/show_bug.cgi?id=65873
Aaron Watry changed:
What|Removed |Added
Attachment #80972|0 |1
is obsolete|
https://bugs.freedesktop.org/show_bug.cgi?id=65873
Aaron Watry changed:
What|Removed |Added
Attachment #80971|0 |1
is obsolete|
https://bugs.freedesktop.org/show_bug.cgi?id=65873
--- Comment #4 from Aaron Watry ---
The good news is that the store error is gone...
Now I get a load error :/
LLVM ERROR: Cannot select: 0x2c92af0: i64,ch = load 0x21fe0c8, 0x2c924f0,
0x2c921f0 [ID=15]
0x2c924f0: i64 = add 0x2c91ef0, 0x2c923
https://bugs.freedesktop.org/show_bug.cgi?id=65873
--- Comment #3 from Tom Stellard ---
These patches should fix the issue:
http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20130617/178156.html
The first patch is unrelated to the bug, but it is required in order to prevent
the test c
https://bugs.freedesktop.org/show_bug.cgi?id=65873
--- Comment #2 from Aaron Watry ---
Created attachment 80972
--> https://bugs.freedesktop.org/attachment.cgi?id=80972&action=edit
Output from llc -debug-only=isel --march=r600 --mcpu=verde < vload-int.ll
--
You are receiving this mail because
https://bugs.freedesktop.org/show_bug.cgi?id=65873
--- Comment #1 from Aaron Watry ---
Created attachment 80971
--> https://bugs.freedesktop.org/attachment.cgi?id=80971&action=edit
LLVM assembly that is produced from the vload-int.cl test case
--
You are receiving this mail because:
You are t
https://bugs.freedesktop.org/show_bug.cgi?id=65873
--- Comment #2 from Aaron Watry ---
Created attachment 80972
--> https://bugs.freedesktop.org/attachment.cgi?id=80972&action=edit
Output from llc -debug-only=isel --march=r600 --mcpu=verde < vload-int.ll
--
You are receiving this mail because
https://bugs.freedesktop.org/show_bug.cgi?id=65873
--- Comment #1 from Aaron Watry ---
Created attachment 80971
--> https://bugs.freedesktop.org/attachment.cgi?id=80971&action=edit
LLVM assembly that is produced from the vload-int.cl test case
--
You are receiving this mail because:
You are t
30 matches
Mail list logo