Closed Bug 1045865 Opened 10 years ago Closed 9 years ago

AlphaBoxBlur needs NEON implementation

Categories

(Core :: Graphics, defect)

x86
macOS
defect
Not set
normal

Tracking

()

RESOLVED DUPLICATE of bug 753010

People

(Reporter: BenWa, Unassigned)

References

Details

Attachments

(2 files)

We have a SSE2 & C implementation of AlphaBoxBlur but we don't have a NEON implementation. AlphaBoxBlur is used heavy by the b2g homescreen and takes over 25% of our rendering time:

http://people.mozilla.org/~bgirard/cleopatra/#report=46f55482af4f58a113e3a4c81ffe24f69fc911f2
Blocks: 1045852
Bug 942023 might be an easier (and possibly more effective) way to fix this.
Attached patch wipSplinter Review
I started on this a few months ago while I was almost asleep at an airport. Here's what I ended up with. I can't recall whether it even builds. But it's probably better than starting from scratch.

Even if we won't be using this for box-shadow, it'll still benefit the SVG/CSS blur filter.
Attached patch Here's my WIPSplinter Review
I also looked at doing this last night. Mine certainly doesn't build yet.
Because two implementations isn't enough, we now have a third implementation in bug 753010. But that one has been tested!
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: