Browse Source

Cleanup

pull/3113/head
James Jackson-South 4 weeks ago
parent
commit
3a879338c6
  1. 10
      src/ImageSharp/Processing/Processors/Convolution/Convolution2PassProcessor{TPixel}.cs
  2. 1
      src/ImageSharp/Processing/Processors/Convolution/ConvolutionProcessor{TPixel}.cs

10
src/ImageSharp/Processing/Processors/Convolution/Convolution2PassProcessor{TPixel}.cs

@ -109,10 +109,6 @@ internal class Convolution2PassProcessor<TPixel> : ImageProcessor<TPixel>
MemoryAllocator allocator = this.Configuration.MemoryAllocator;
// Convolution is memory-bandwidth-bound with low arithmetic intensity.
// Parallelization degrades performance due to cache line contention from
// overlapping source row reads. See #3111.
// Horizontal convolution
HorizontalConvolutionRowOperation horizontalOperation = new(
interest,
@ -124,9 +120,9 @@ internal class Convolution2PassProcessor<TPixel> : ImageProcessor<TPixel>
this.PreserveAlpha);
ParallelRowIterator.IterateRows<HorizontalConvolutionRowOperation, Vector4>(
this.Configuration,
interest,
in horizontalOperation);
this.Configuration,
interest,
in horizontalOperation);
// Vertical convolution
VerticalConvolutionRowOperation verticalOperation = new(

1
src/ImageSharp/Processing/Processors/Convolution/ConvolutionProcessor{TPixel}.cs

@ -1,7 +1,6 @@
// Copyright (c) Six Labors.
// Licensed under the Six Labors Split License.
using System.Buffers;
using System.Numerics;
using System.Runtime.CompilerServices;
using System.Runtime.InteropServices;

Loading…
Cancel
Save