Skip to content
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

Bug: Build optimization warning #333

Open
2 tasks done
sroussey opened this issue Jan 7, 2024 · 3 comments
Open
2 tasks done

Bug: Build optimization warning #333

sroussey opened this issue Jan 7, 2024 · 3 comments
Assignees
Labels
good first issue Good for newcomers invalid This doesn't seem right

Comments

@sroussey
Copy link
Contributor

sroussey commented Jan 7, 2024

Describe the bug

It is a warning "loop not vectorized" which normally I would ignore, but since this lib is all about vectorizing for speed I think it is unexpected.

Steps to reproduce

npm install

Expected behavior

no warning. Instead I get this:

warning: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/libtool: archive library: Release/nothing.a the table of contents is empty (no object file members in the library define global symbols)
  CXX(target) Release/obj.target/usearch/javascript/lib.o
In file included from ../javascript/lib.cpp:18:
In file included from ../include/usearch/index_dense.hpp:10:
../include/usearch/index_plugins.hpp:1208:9: warning: loop not vectorized: the optimizer was unable to perform the requested transformation; the transformation might be disabled or specified as part of an unsupported transformation ordering [-Wpass-failed=transform-warning]
        for (std::size_t i = 0; i != dim; ++i) {

USearch version

main

Operating System

MacOS 14.2.1 / Darwin 23.2.0 arm64 arm

Hardware architecture

Arm

Which interface are you using?

C++ implementation

Contact Details

No response

Is there an existing issue for this?

  • I have searched the existing issues

Code of Conduct

  • I agree to follow this project's Code of Conduct
@sroussey sroussey added bug Something isn't working invalid This doesn't seem right labels Jan 7, 2024
@ashvardanian
Copy link
Contributor

Yes, that's a common warning. I think the CMakeLists.txt, setup.py and bindings.gyp should be updated to add the -Wpass-failed=transform-warning flag for warnings, as mentioned in your log. @sroussey does that help? Would you be able to open a PR? Thanks 🤗

@sroussey
Copy link
Contributor Author

I added that to bindings.gyp and no change.

@sroussey
Copy link
Contributor Author

It is already in CMakeLists.txt

@ashvardanian ashvardanian removed the bug Something isn't working label Jan 10, 2024
@ashvardanian ashvardanian self-assigned this Jan 10, 2024
@ashvardanian ashvardanian added the good first issue Good for newcomers label Feb 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers invalid This doesn't seem right
Projects
None yet
Development

No branches or pull requests

2 participants