WebKit Bugzilla
New
Browse
Search+
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
NEW
263874
Regression(
244527@main
): SVG lightning filter for ARM-NEON
https://bugs.webkit.org/show_bug.cgi?id=263874
Summary
Regression(244527@main): SVG lightning filter for ARM-NEON
Przemyslaw Gorszkowski
Reported
2023-10-30 03:39:48 PDT
It seems that in
https://github.com/WebKit/WebKit/commit/58c68bcae31b110160f93fe60e42387dbf6009ae
was done some refactoring things and during that the implementation for the ARM-NEON was removed. Specifically calling function: platformArithmeticNeon in FEComposite::platformArithmeticSoftware. The functionality from the FEComposite::platformArithmeticSoftware was moved to FECompositeSoftwareApplier::applyArithmetic but without distinguished separate functionality for HAVE(ARM_NEON_INTRINSICS) and not HAVE(ARM_NEON_INTRINSICS). There were future fixes for case not HAVE(ARM_NEON_INTRINSICS) (Build error in ARMv7 Neon targets:
https://github.com/WebKit/WebKit/commit/a318e830f46865c9f1fdea4e8cd1132891210b08
) but there is still missing calling functionality for HAVE(ARM_NEON_INTRINSICS). The problem was observed and reported for WPE 2.38:
https://github.com/WebPlatformForEmbedded/WPEWebKit/issues/1145
as a regression in case of some tests of SVG lightning filters. There is also fix for that:
https://github.com/WebPlatformForEmbedded/WPEWebKit/pull/1199
which can be reused in upstream to fix that (SVG lightning filters for ARM-NEON platforms).
Attachments
Add attachment
proposed patch, testcase, etc.
Przemyslaw Gorszkowski
Comment 1
2023-10-30 23:24:33 PDT
Pull request:
https://github.com/WebKit/WebKit/pull/19762
Radar WebKit Bug Importer
Comment 2
2023-11-06 02:40:14 PST
<
rdar://problem/117990832
>
Ahmad Saleem
Comment 3
2024-05-21 05:50:04 PDT
@Should this be closed since PR referred to this commit - which has now landed -
https://github.com/WebKit/WebKit/pull/2926
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug