Skip to content

[0.6] Beng arm opt f64 - [MOD-9077] #648

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 9 commits into from
Apr 9, 2025
Merged

Conversation

BenGoldberger
Copy link
Collaborator

@BenGoldberger BenGoldberger commented Apr 9, 2025

Backport of #618 to 0.6.

BenGoldberger and others added 7 commits April 9, 2025 11:43
* Add arm support

* Changed the arm cpu info

* Add ip test

* Add to tests

* Added tests andbm

* fix tests

* Add github benchmakrs

* Check 1

* only arm

* change ami

* Try ireland

* Try different image

* try image

* back to old image

* larger image

* Add option to change env

* back to default region

* Created new image

* Try to add the x86 to check

* Try different machine

* added include

* Try without opti on arm

* Change to c6g

* added matrix region

* change to west

* try the i8

* Try oregon

* Change subnet id

* Now subnet

* Change subnet

* add subnet

* Try group id

* Change to vpc id

* change subnet

* Change ami

* Try without subnet

* add security group again

* Change the subnets

* Change to ids

* Change sg

* psubnet

* Try different

* different

* to a file

* print

* p

* leave empty

* empty

* Try different account

* Run 2 arm machines

* Move both to us-west-2

* Try workflow

* Change name

* Changes

* Change the secrets

* Add supprted arch

* Add defaults

* Support all

* Change the jq

* Change machine to t4g

* Change the name

* Change the machine

* fix the stop

* only benchamrk

* add the secrets

* region secret

* benchmark region

* Change timeout

* Added support for arch name in benchamrks

* change th json

* changed to v9.0

* Change the check

* add v9

* Check alt version of armv9

* added check

* add arc_arch

* changed to CONCAT_WITH_UNDERSCORE_ARCH

* change the check

* Add full check

* fix the instruct

* Added the cmake

* fix the support

* put it back to cmake

* back

* change the condition

* No armpl for now

* cland format

* remove the opt

* Changed to one machine

* Added BENCHMARK_ARCH

* fix endif

* Remove secrets call

* pr changes

* Changes

* change to compile

* add sve

* add #endif

* add armpl

* add to cmake

* remove armpl

* add install

* Add ARCH=$(uname -m)

* change the path to armpl

* suuport check for armv7

* change the armpl

* Change or OR

* add neon supported for spaces

* add sve

* add support

* align

* format

* change error

* change

* Removed the ifdef

* Add comments

* clang

* Change names

* format

* Try fp32 neon simd

* add l2

* add cmake

* add SVE

* fix sve l2

* PR changes

* Change to 1

* fix the l2

* fix format

* F64 ARMPL optimizations for SVE and NEON

* add desciriopn for chunk == 1

* Change functions

* Add include

* Change the cast

* add resudual

* formatting

* Move th consexpt

* remove template armpl

* Back to armpl

* back to armpl_neon

* include

* armnpl

* add choose

* fix the residual div

* raise the residuals values

* back to char

* add ip and l2 intrinsic opt

* changes from f32 to f64

* changes to match f32

* small fixes

* small fixes

* Remove prefetch

* Revert implemetion chooser

* Remove armpl

* Revert remove error

* Remove comment

* Remove empty line

* try to fix neo f64

* try to get arm opt to work

* try to get arm opt to work

* try to get arm opt to work

* try to get arm opt to work

* format

* removed files

* remove armpl

* Add support macos

* add sudo

* Add absolute path

* find all libs

* Change folder

* Now set for real

* try to get arm opt to work

* Remove armpl from pull

* try to get arm opt to work

* try to get arm opt to work

* try to get arm opt to work

* try to get arm opt to work

* try to test fp64 opt

* try to test fp64 opt

* fix sve2 implementation

* fix fp64 opt test

* fix fp64 opt test

* change the templates

* change chunk size to 1

* Back to 4

* chenge to comply with fp32 implementation

* fix sve2

* fix ip template

* small fix

* Removed the for

* Change to 2 sums

* Changed

* Add get opt func

* Change the var name

* format

* Pr fixes

* chage implementation to match fp32

* small fix

* small fix

* small fix

* samll fix

* changes to test

* small fix

* format changes

* PR

* pr changes

* pr

* try to fix ci tests

* pr fix

* PR

* added conversion

* small dim for intel only

* Test smallDimChooser only for intel

* remove sve2 h files

* pr changes

* pr changes

* pr changes

* pr changes

* pr changes

* pr changes

* pr changes

* format changes

* casting for mac os

---------

Co-authored-by: Dor Forer <dor.forer@redis.com>
@BenGoldberger BenGoldberger marked this pull request as ready for review April 9, 2025 12:35
@BenGoldberger BenGoldberger requested a review from dor-forer April 9, 2025 12:35
dor-forer
dor-forer previously approved these changes Apr 9, 2025
Copy link
Collaborator

@dor-forer dor-forer left a comment

Choose a reason for hiding this comment

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

Just update the pr name and description

Copy link
Collaborator

Choose a reason for hiding this comment

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

What is that?

@BenGoldberger BenGoldberger changed the title Backport-618-to-0.6 [0.6] Beng arm opt f64 - [MOD-9077] Apr 9, 2025
Copy link

codecov bot commented Apr 9, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.02%. Comparing base (82f51e9) to head (ef99709).
Report is 1 commits behind head on 0.6.

Additional details and impacted files
@@           Coverage Diff           @@
##              0.6     #648   +/-   ##
=======================================
  Coverage   95.02%   95.02%           
=======================================
  Files          60       60           
  Lines        3418     3418           
=======================================
  Hits         3248     3248           
  Misses        170      170           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@BenGoldberger BenGoldberger added this pull request to the merge queue Apr 9, 2025
Merged via the queue into 0.6 with commit 3e58052 Apr 9, 2025
17 checks passed
@BenGoldberger BenGoldberger deleted the backport-618-to-0.6 branch April 9, 2025 13:57
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.

2 participants