Skip to content

Add ARM NEON intrinsics to align - #13389

Merged
Nir-Az merged 1 commit into
realsenseai:developmentfrom
fateshelled:neon-align
Oct 13, 2024
Merged

Add ARM NEON intrinsics to align#13389
Nir-Az merged 1 commit into
realsenseai:developmentfrom
fateshelled:neon-align

Conversation

@fateshelled

Copy link
Copy Markdown
Contributor

Changes:

  • adding align_neon class
    • neon-align.cpp is based on the code in sse-align.cpp
    • tested on Ubuntu 22.04, OrangePi5 (RK3588s, 8GB of RAM) and RealSense D435.

I optimized depth align processing for ARM CPUs.
Please review it.

@sysrsbuild

Copy link
Copy Markdown
Contributor

Can one of the admins verify this patch?

Comment thread src/proc/neon/neon-align.h Outdated
// License: Apache 2.0. See LICENSE file in root directory.
// Copyright(c) 2024 Intel Corporation. All Rights Reserved.

#ifndef LIBREALSENSE_NEON_ALIGN_H

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you replace to :
#pragma once ?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

replaced it.

@Nir-Az

Nir-Az commented Oct 13, 2024

Copy link
Copy Markdown
Collaborator

@fateshelled I merged the other PR and now we have a conflict.
Please rebase and see if fixed.
Thanks

@fateshelled

Copy link
Copy Markdown
Contributor Author

@Nir-Az Thanks for reviewing. I rebased and fixed it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants