Skip to content

Conversation

@fateshelled
Copy link
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
Contributor

Can one of the admins verify this patch?

// 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
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
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
Copy link
Collaborator

Nir-Az commented Oct 13, 2024

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

@fateshelled
Copy link
Contributor Author

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

@Nir-Az Nir-Az merged commit 4b1a208 into realsenseai:development Oct 13, 2024
19 checks passed
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