Bug 263874 - Regression(244527@main): SVG lightning filter for ARM-NEON
Summary: Regression(244527@main): SVG lightning filter for ARM-NEON
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: WebKit Nightly Build
Hardware: Other Linux
: P2 Normal
Assignee: Przemyslaw Gorszkowski
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2023-10-30 03:39 PDT by Przemyslaw Gorszkowski
Modified: 2024-05-21 05:50 PDT (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Przemyslaw Gorszkowski 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).
Comment 1 Przemyslaw Gorszkowski 2023-10-30 23:24:33 PDT
Pull request: https://github.com/WebKit/WebKit/pull/19762
Comment 2 Radar WebKit Bug Importer 2023-11-06 02:40:14 PST
<rdar://problem/117990832>
Comment 3 Ahmad Saleem 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