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

Add neon_cnt and neon_addv8 for 64-bit ARM for parity with MSVC #50830

Closed
llvmbot opened this issue Aug 16, 2021 · 1 comment · Fixed by microsoft/STL#4695
Closed

Add neon_cnt and neon_addv8 for 64-bit ARM for parity with MSVC #50830

llvmbot opened this issue Aug 16, 2021 · 1 comment · Fixed by microsoft/STL#4695
Labels
backend:AArch64 bugzilla Issues migrated from bugzilla c++ clang:headers Headers provided by Clang, e.g. for intrinsics obsolete Issues with old (unsupported) versions of LLVM

Comments

@llvmbot
Copy link
Collaborator

llvmbot commented Aug 16, 2021

Bugzilla Link 51488
Version 12.0
OS Windows NT
Reporter LLVM Bugzilla Contributor
CC @DougGregor,@zygoloid

Extended Description

Hello.
I started to work on popcount optimization on ARM64: microsoft/STL#2127
and I found linking errors with clang-cl.
Could you provide neon_cnt and neon_addv8? (Also neon_cntq, neon_addvq8 for future optimizing bitset: microsoft/STL#2129 (comment))

__n64  neon_cnt(__n64);
__n128 neon_cntq(__n128);
__n8  neon_addv8(__n64);
__n8  neon_addvq8(__n128);

Or do you already provide similar intrinsics with different names?
I am slightly worried that they are not _Ugly.
So maybe I should ask Microsoft as well, maybe they should rename it if you already have it with suitable names...

I used Windows 10 x86-64, Visual Studio 2022 Preview 3, Clang 12 from Visual Studio.

@llvmbot llvmbot transferred this issue from llvm/llvm-bugzilla-archive Dec 11, 2021
@RKSimon RKSimon added clang:headers Headers provided by Clang, e.g. for intrinsics backend:AArch64 labels Dec 31, 2021
@StephanTLavavej
Copy link
Member

MSVC's STL is now using different intrinsics, so we've closed this issue.

@EugeneZelenko EugeneZelenko added the obsolete Issues with old (unsupported) versions of LLVM label Jun 18, 2024
@EugeneZelenko EugeneZelenko closed this as not planned Won't fix, can't repro, duplicate, stale Jun 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend:AArch64 bugzilla Issues migrated from bugzilla c++ clang:headers Headers provided by Clang, e.g. for intrinsics obsolete Issues with old (unsupported) versions of LLVM
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants