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.

Sign up for GitHub

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

Jump to bottom

Inline _byteswap_* functions for optimized build #39999

Closed
llvmbot opened this issue Feb 8, 2019 · 0 comments
Closed

Inline _byteswap_* functions for optimized build #39999

llvmbot opened this issue Feb 8, 2019 · 0 comments
Labels
bugzilla Issues migrated from bugzilla clang:frontend Language frontend issues, e.g. anything involving "Sema"

Comments

@llvmbot
Copy link
Collaborator

llvmbot commented Feb 8, 2019

Bugzilla Link 40653
Version unspecified
OS Windows NT
Reporter LLVM Bugzilla Contributor
CC @zygoloid

Extended Description

byteswap* series functions (_byteswap_uint64, _byteswap_ulong, _byteswap_ushort) are provided as library functions in libucrt.lib, but MSVC doesn't inline them for optimized build. It will be great to inline them in clang-cl.exe for optimized build.
@llvmbot llvmbot transferred this issue from llvm/llvm-bugzilla-archive Dec 10, 2021
mem-frob pushed a commit to draperlaboratory/hope-llvm-project that referenced this issue Oct 7, 2022
@efriedma-quic
[CodeGen] Inline _byteswap_* builtins.
b04b8d3 
As discussed in D57915.

Fixes llvm/llvm-project#39999 .

Differential Revision: https://reviews.llvm.org/D121865
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bugzilla Issues migrated from bugzilla clang:frontend Language frontend issues, e.g. anything involving "Sema"
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@llvmbot