[Pixman] [PATCH] MIPS: DSPr2: Fix bug in over_n_8888_8888_ca/over_n_8888_0565_ca routines
siarhei.siamashka at gmail.com
Wed Mar 6 11:57:14 PST 2013
On Mon, 4 Mar 2013 10:58:42 +0100
"Nemanja Lukic" <nemanja.lukic at rt-rk.com> wrote:
> > Are you referring to MIPS implementation of the following code?
> > Looks like a lot of changes for only adding a missing shift. Are you
> > really just fixing a single bug and not also introducing something
> > unrelated?
> Yes, it really does look like a huge change for couple of missing shifts.
> When I wrote this code in the first place, I misplaced those shifts, which
> allowed me to combine code for over operation and:
> UN8x4_MUL_UN8x4 (s, ma);
> UN8x4_MUL_UN8 (ma, srca);
> ma = ~ma;
> UN8x4_MUL_UN8x4_ADD_UN8x4 (d, ma, s);
> where shifts are not present (for ma). So I decided to rewrite that piece of
> from scratch. I changed logic, so now assembly code mimic code from
> but process two pixels at a time. This code should be easier to debug and
OK, thanks. It would be nice to have this explanation in the commit
Or alternatively, fixing the bug in one commit and then reorganizing
the code to make it cleaner/faster in another commit would have been
much easier to review.
More information about the Pixman