diff options
| author | Michele Di Giorgio <michele.digiorgio@arm.com> | 2021-03-30 12:48:35 +0100 |
|---|---|---|
| committer | Michele Di Giorgio <michele.digiorgio@arm.com> | 2021-03-31 17:08:39 +0000 |
| commit | 5f91b5d7063462854b62d342f9d4e04ae647e9a6 (patch) | |
| tree | b15540fd14435cb3c66e77be01ca22d970b5e70f | |
| parent | c6f9510bcb754afaadfe9477ff85d6c55ffcf43b (diff) | |
| download | ComputeLibrary-5f91b5d7063462854b62d342f9d4e04ae647e9a6.tar.gz | |
Remove alignment FIXME
COMPMID-1088 has been closed and the FIXME can be removed.
Change-Id: I2ee103ab12e65383a62bfe3fc4aa0ed90c211510
Signed-off-by: Michele Di Giorgio <michele.digiorgio@arm.com>
Reviewed-on: https://review.mlplatform.org/c/ml/ComputeLibrary/+/5341
Reviewed-by: Georgios Pinitas <georgios.pinitas@arm.com>
Comments-Addressed: Arm Jenkins <bsgcomp@arm.com>
Tested-by: Arm Jenkins <bsgcomp@arm.com>
| -rw-r--r-- | src/runtime/NEON/functions/NEGEMMAssemblyDispatch.cpp | 6 | ||||
| -rw-r--r-- | src/runtime/cpu/operators/CpuPoolingAssemblyDispatch.cpp | 2 |
2 files changed, 4 insertions, 4 deletions
diff --git a/src/runtime/NEON/functions/NEGEMMAssemblyDispatch.cpp b/src/runtime/NEON/functions/NEGEMMAssemblyDispatch.cpp index 8114adadc2..e72a6c3226 100644 --- a/src/runtime/NEON/functions/NEGEMMAssemblyDispatch.cpp +++ b/src/runtime/NEON/functions/NEGEMMAssemblyDispatch.cpp @@ -182,7 +182,7 @@ public: void configure(size_t B_pretranspose_size, unsigned int alignment) { - _output.allocator()->init(TensorInfo(TensorShape{ (B_pretranspose_size + alignment /* FIXME: remove alignment after COMPMID-1088 */) }, 1, DataType::S8), alignment); + _output.allocator()->init(TensorInfo(TensorShape{ (B_pretranspose_size + alignment) }, 1, DataType::S8), alignment); _B_pretranspose_size = B_pretranspose_size; } @@ -525,7 +525,7 @@ void Fallback<TypeInput, TypeOutput, OutputStage>::configure(const ITensor *a, c else { _pretranspose = new Tensor(); - static_cast<Tensor *>(_pretranspose)->allocator()->init(TensorInfo(TensorShape{ (B_pretranspose_size + alignment /* FIXME: remove alignment after COMPMID-1088 */) }, 1, DataType::S8), alignment); + static_cast<Tensor *>(_pretranspose)->allocator()->init(TensorInfo(TensorShape{ (B_pretranspose_size + alignment) }, 1, DataType::S8), alignment); } } @@ -587,7 +587,7 @@ template <typename TypeInput, typename TypeOutput, class OutputStage> void Fallback<TypeInput, TypeOutput, OutputStage>::allocate_workspace(size_t workspace_size, MemoryGroup &memory_group, size_t alignment) { ARM_COMPUTE_ERROR_ON_MSG(workspace_size == 0, "size cannot be 0"); - _workspace.allocator()->init(TensorInfo(TensorShape{ (workspace_size + alignment /* FIXME: remove alignment after COMPMID-1088 */) }, 1, DataType::S8), alignment); + _workspace.allocator()->init(TensorInfo(TensorShape{ (workspace_size + alignment) }, 1, DataType::S8), alignment); memory_group.manage(&_workspace); _workspace.allocator()->allocate(); } diff --git a/src/runtime/cpu/operators/CpuPoolingAssemblyDispatch.cpp b/src/runtime/cpu/operators/CpuPoolingAssemblyDispatch.cpp index 4a5623394f..e4526c5bd3 100644 --- a/src/runtime/cpu/operators/CpuPoolingAssemblyDispatch.cpp +++ b/src/runtime/cpu/operators/CpuPoolingAssemblyDispatch.cpp @@ -62,7 +62,7 @@ void CpuPoolingAssemblyDispatch::configure(const ITensorInfo *src, ITensorInfo * // Allocate workspace based on kernel's memory requirements constexpr size_t alignment = 4096; const size_t workspace_size = pooling_wrapper->get_working_size(num_threads); - _workspace.allocator()->init(TensorInfo(TensorShape{ (workspace_size + alignment /* FIXME: remove alignment after COMPMID-1088 */) }, 1, DataType::S8), alignment); + _workspace.allocator()->init(TensorInfo(TensorShape{ (workspace_size + alignment) }, 1, DataType::S8), alignment); _memory_group.manage(&_workspace); _workspace.allocator()->allocate(); |