Diff
Modified: trunk/Source/WebCore/ChangeLog (288558 => 288559)
--- trunk/Source/WebCore/ChangeLog 2022-01-25 17:18:35 UTC (rev 288558)
+++ trunk/Source/WebCore/ChangeLog 2022-01-25 17:34:49 UTC (rev 288559)
@@ -1,3 +1,18 @@
+2022-01-25 Pablo Saavedra <psaave...@igalia.com>
+
+ [WPE][GTK] Build error in ARMv7 Neon targets after r286152
+ https://bugs.webkit.org/show_bug.cgi?id=235410
+
+ Reviewed by Simon Fraser.
+
+ * platform/graphics/cpu/arm/filters/FEBlendNEON.h:
+ (WebCore::FEBlend::platformApplySoftware): Deleted.
+ * platform/graphics/filters/software/FEBlendSoftwareApplier.cpp:
+ (WebCore::FEBlendSoftwareApplier::apply const):
+ * platform/graphics/filters/software/FECompositeSoftwareApplier.cpp:
+ (WebCore::FECompositeSoftwareApplier::applyArithmetic const):
+ * platform/graphics/filters/software/FEGaussianBlurSoftwareApplier.cpp:
+
2022-01-25 Antti Koivisto <an...@apple.com>
[CSS Container Queries] Parsing support for container-name property
Modified: trunk/Source/WebCore/platform/graphics/cpu/arm/filters/FEBlendNEON.h (288558 => 288559)
--- trunk/Source/WebCore/platform/graphics/cpu/arm/filters/FEBlendNEON.h 2022-01-25 17:18:35 UTC (rev 288558)
+++ trunk/Source/WebCore/platform/graphics/cpu/arm/filters/FEBlendNEON.h 2022-01-25 17:34:49 UTC (rev 288559)
@@ -106,42 +106,6 @@
}
};
-bool FEBlend::platformApplySoftware(const Filter&)
-{
- FilterEffect* in = inputEffect(0);
- FilterEffect* in2 = inputEffect(1);
-
- auto& destinationPixelBuffer = pixelBufferResult(AlphaPremultiplication::Premultiplied);
- if (!destinationPixelBuffer)
- return false;
-
- auto& destinationPixelArray = destinationPixelBuffer->data();
-
- IntRect effectADrawingRect = requestedRegionOfInputPixelBuffer(in->absolutePaintRect());
- auto sourcePixelArrayA = in->getPixelBufferResult(AlphaPremultiplication::Premultiplied, effectADrawingRect);
-
- IntRect effectBDrawingRect = requestedRegionOfInputPixelBuffer(in2->absolutePaintRect());
- auto sourcePixelArrayB = in2->getPixelBufferResult(AlphaPremultiplication::Premultiplied, effectBDrawingRect);
-
- unsigned sourcePixelArrayLength = sourcePixelArrayA->length();
- ASSERT(pixelArrayLength == sourcePixelArrayB->length());
-
- if (sourcePixelArrayLength >= 8) {
- platformApplyNEON(sourcePixelArrayA->data(), sourcePixelArrayB->data(), destinationPixelArray.data(), sourcePixelArrayLength);
- return true;
- }
- // If there is just one pixel we expand it to two.
- ASSERT(sourcePixelArrayLength > 0);
- uint32_t sourceA[2] = {0, 0};
- uint32_t sourceBAndDest[2] = {0, 0};
-
- sourceA[0] = reinterpret_cast<uint32_t*>(sourcePixelArrayA->data())[0];
- sourceBAndDest[0] = reinterpret_cast<uint32_t*>(sourcePixelArrayB->data())[0];
- platformApplyNEON(reinterpret_cast<uint8_t*>(sourceA), reinterpret_cast<uint8_t*>(sourceBAndDest), reinterpret_cast<uint8_t*>(sourceBAndDest), 8);
- reinterpret_cast<uint32_t*>(destinationPixelArray.data())[0] = sourceBAndDest[0];
- return true;
-}
-
void FEBlend::platformApplyNEON(unsigned char* srcPixelArrayA, unsigned char* srcPixelArrayB, unsigned char* dstPixelArray,
unsigned colorArrayLength)
{
Modified: trunk/Source/WebCore/platform/graphics/filters/software/FEBlendSoftwareApplier.cpp (288558 => 288559)
--- trunk/Source/WebCore/platform/graphics/filters/software/FEBlendSoftwareApplier.cpp 2022-01-25 17:18:35 UTC (rev 288558)
+++ trunk/Source/WebCore/platform/graphics/filters/software/FEBlendSoftwareApplier.cpp 2022-01-25 17:34:49 UTC (rev 288559)
@@ -33,7 +33,6 @@
namespace WebCore {
-#if !HAVE(ARM_NEON_INTRINSICS)
bool FEBlendSoftwareApplier::apply(const Filter&, const FilterImageVector& inputs, FilterImage& result) const
{
auto& input = inputs[0].get();
@@ -56,6 +55,5 @@
filterContext.drawImageBuffer(*inputImage, inputImageRect, { { }, inputImage->logicalSize() }, { CompositeOperator::SourceOver, m_effect.blendMode() });
return true;
}
-#endif
} // namespace WebCore
Modified: trunk/Source/WebCore/platform/graphics/filters/software/FECompositeSoftwareApplier.cpp (288558 => 288559)
--- trunk/Source/WebCore/platform/graphics/filters/software/FECompositeSoftwareApplier.cpp 2022-01-25 17:18:35 UTC (rev 288558)
+++ trunk/Source/WebCore/platform/graphics/filters/software/FECompositeSoftwareApplier.cpp 2022-01-25 17:34:49 UTC (rev 288559)
@@ -145,7 +145,9 @@
int length = sourcePixelArray.length();
ASSERT(length == static_cast<int>(destinationPixelArray.length()));
+#if !HAVE(ARM_NEON_INTRINSICS)
applyPlatformArithmetic(sourcePixelArray.data(), destinationPixelArray.data(), length, m_effect.k1(), m_effect.k2(), m_effect.k3(), m_effect.k4());
+#endif
return true;
}
Modified: trunk/Source/WebCore/platform/graphics/filters/software/FEGaussianBlurSoftwareApplier.cpp (288558 => 288559)
--- trunk/Source/WebCore/platform/graphics/filters/software/FEGaussianBlurSoftwareApplier.cpp 2022-01-25 17:18:35 UTC (rev 288558)
+++ trunk/Source/WebCore/platform/graphics/filters/software/FEGaussianBlurSoftwareApplier.cpp 2022-01-25 17:34:49 UTC (rev 288559)
@@ -27,6 +27,9 @@
#include "FEGaussianBlurSoftwareApplier.h"
#include "FEGaussianBlur.h"
+#if HAVE(ARM_NEON_INTRINSICS)
+#include "FEGaussianBlurNEON.h"
+#endif
#include "GraphicsContext.h"
#include "ImageBuffer.h"
#include "PixelBuffer.h"
