pixman: Branch 'master' - 2 commits
Jeff Muizelaar
jrmuizel at kemper.freedesktop.org
Thu Jun 4 09:34:20 PDT 2009
pixman/pixman-arm-neon.c | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
New commits:
commit 5f086792eeaea6b2c401105b8bbf0c92fb7d192e
Author: Jonathan Morton <jmorton at sd070.hel.movial.fi>
Date: Thu Jun 4 11:24:26 2009 -0400
[NEON] Really fix filler bug.
Advance the destination pointer (r4 register) properly.
Found by Siarhei Siamashka.
diff --git a/pixman/pixman-arm-neon.c b/pixman/pixman-arm-neon.c
index 5d07c76..abcd24f 100644
--- a/pixman/pixman-arm-neon.c
+++ b/pixman/pixman-arm-neon.c
@@ -1703,15 +1703,15 @@ pixman_fill_neon (uint32_t *bits,
"2:\n"
"tst %[width], #4\n"
"beq 2f\n"
- "str %[color], [r4]!\n"
+ "str %[color], [r4], #4\n"
"2:\n"
"tst %[width], #2\n"
"beq 2f\n"
- "strh %[color], [r4]!\n"
+ "strh %[color], [r4], #2\n"
"2:\n"
"tst %[width], #1\n"
"beq 2f\n"
- "strb %[color], [r4]!\n"
+ "strb %[color], [r4], #1\n"
"2:\n"
"subs %[height], %[height], #1\n"
commit 3b12cc7a23f81581b027764be96bf028785b1b5f
Author: Jonathan Morton <jmorton at sd070.hel.movial.fi>
Date: Thu Jun 4 11:24:22 2009 -0400
[NEON] Fix filler bug.
r5 is being sourced explicitly instead of the %[width] reference.
It's probably a copy-paste bug, not spotted because I didn't
originally write it.
diff --git a/pixman/pixman-arm-neon.c b/pixman/pixman-arm-neon.c
index 36d7026..5d07c76 100644
--- a/pixman/pixman-arm-neon.c
+++ b/pixman/pixman-arm-neon.c
@@ -1709,7 +1709,7 @@ pixman_fill_neon (uint32_t *bits,
"beq 2f\n"
"strh %[color], [r4]!\n"
"2:\n"
- "tst r5, #1\n"
+ "tst %[width], #1\n"
"beq 2f\n"
"strb %[color], [r4]!\n"
"2:\n"
More information about the xorg-commit
mailing list