RESOLVED INVALID Bug 233500
Remove the code under #if HAVE(ARM_NEON_INTRINSICS) 
https://bugs.webkit.org/show_bug.cgi?id=233500
Summary Remove the code under #if HAVE(ARM_NEON_INTRINSICS)
Said Abou-Hallawa
Reported 2021-11-25 13:21:42 PST
The sources and headers under platform/graphics/cpu/arm/ do not compile anymore. 1. The file platform/graphics/cpu/arm/filters/FEBlendNEON.h won't compile because it references FilterEffect::pixelBufferResult() which does not exist anymore. 2. The file platform/graphics/cpu/arm/filters/FELightingNEON.cpp won't compile because it references FELighting:LightingData which does not exist anymore.
Attachments
Patch (94.99 KB, patch)
2021-11-25 13:28 PST, Said Abou-Hallawa 		no flags
DetailsFormatted DiffDiff
Patch (96.21 KB, patch)
2021-11-25 15:11 PST, Said Abou-Hallawa 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Said Abou-Hallawa
Comment 1 2021-11-25 13:28:51 PST
Created attachment 445153 [details] Patch
Said Abou-Hallawa
Comment 2 2021-11-25 15:11:27 PST
Created attachment 445158 [details] Patch
Said Abou-Hallawa
Comment 3 2021-11-25 23:27:33 PST
These header files were not referenced in the code: platform/graphics/cpu/arm/filters/FECompositeArithmeticNEON.h platform/graphics/cpu/arm/filters/FEGaussianBlurNEON.h
Radar WebKit Bug Importer
Comment 4 2021-12-02 13:22:21 PST
<rdar://problem/85986173>
Said Abou-Hallawa
Comment 5 2022-01-25 14:23:17 PST
Bug 235410 is the opposite of this bug. It tries to get the code under HAVE(ARM_NEON_INTRINSICS) be compiled. So close this one for now. My understanding is a new bot for armv7 might be added to WebKit build system soon.
Note You need to log in before you can comment on or make changes to this bug.