Skip to content

Incorrect CPUID feature name in the Intrinsics Guide: FP16C instead of F16C #738

New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom
Closed
gnzlbg opened this issue Apr 23, 2019 · 1 comment
Closed

Incorrect CPUID feature name in the Intrinsics Guide: FP16C instead of F16C #738

gnzlbg opened this issue Apr 23, 2019 · 1 comment

Comments

@gnzlbg
Copy link
Contributor

gnzlbg commented Apr 23, 2019

The Intel Intrinsics Guide for the 16-bit half-precision float intrinsics calls the CPUID feature flag "FP16C", see: https://software.intel.com/sites/landingpage/IntrinsicsGuide/#techs=Other&text=fp16&expand=1711

LLVM (e.g. ), CPUID Wikipedia page (https://en.wikipedia.org/wiki/CPUID), and thIntel® 64 and IA-32 Architectures Software Developer’s Manual Volume 2 (2A, 2B, 2C & 2D): Instruction Set Reference, A-Z (https://www.intel.com/content/dam/www/public/us/en/documents/manuals/64-ia-32-architectures-software-developer-instruction-set-reference-manual-325383.pdf) all call this feature "F16C" instead.

This is probably a bug in the Intrinsics Guide that we should report and track.
@gnzlbg gnzlbg mentioned this issue Apr 23, 2019
Merged
Centril added a commit to Centril/rust that referenced this issue Apr 23, 2019
@Centril
Rollup merge of rust-lang#60191 - gnzlbg:f16c, r=alexcrichton
9246397 
Add f16c target_feature

This is requires for Intel 16-bit half-precision float intrinsics: https://software.intel.com/sites/landingpage/IntrinsicsGuide/#text=fp16&expand=1769 - see companion stdsimd PR: rust-lang/stdarch#737.

LLVM, Wikipedia CPUID page, and the Intel Dev Manual all call this CPUID feature "F16C", but the Intel intrinsics guide calls this "FP16C" - this is probably a bug in the intrinsics guide which we are tracking here: rust-lang/stdarch#738

r? @alexcrichton
Centril added a commit to Centril/rust that referenced this issue Apr 23, 2019
@Centril
Rollup merge of rust-lang#60191 - gnzlbg:f16c, r=alexcrichton
97b74a5 
Add f16c target_feature

This is requires for Intel 16-bit half-precision float intrinsics: https://software.intel.com/sites/landingpage/IntrinsicsGuide/#text=fp16&expand=1769 - see companion stdsimd PR: rust-lang/stdarch#737.

LLVM, Wikipedia CPUID page, and the Intel Dev Manual all call this CPUID feature "F16C", but the Intel intrinsics guide calls this "FP16C" - this is probably a bug in the intrinsics guide which we are tracking here: rust-lang/stdarch#738

r? @alexcrichton
Centril added a commit to Centril/rust that referenced this issue Apr 23, 2019
@Centril
Rollup merge of rust-lang#60191 - gnzlbg:f16c, r=alexcrichton
0353fa5 
Add f16c target_feature

This is requires for Intel 16-bit half-precision float intrinsics: https://software.intel.com/sites/landingpage/IntrinsicsGuide/#text=fp16&expand=1769 - see companion stdsimd PR: rust-lang/stdarch#737.

LLVM, Wikipedia CPUID page, and the Intel Dev Manual all call this CPUID feature "F16C", but the Intel intrinsics guide calls this "FP16C" - this is probably a bug in the intrinsics guide which we are tracking here: rust-lang/stdarch#738

r? @alexcrichton
Centril added a commit to Centril/rust that referenced this issue Apr 23, 2019
@Centril
Rollup merge of rust-lang#60191 - gnzlbg:f16c, r=alexcrichton
cd3256f 
Add f16c target_feature

This is requires for Intel 16-bit half-precision float intrinsics: https://software.intel.com/sites/landingpage/IntrinsicsGuide/#text=fp16&expand=1769 - see companion stdsimd PR: rust-lang/stdarch#737.

LLVM, Wikipedia CPUID page, and the Intel Dev Manual all call this CPUID feature "F16C", but the Intel intrinsics guide calls this "FP16C" - this is probably a bug in the intrinsics guide which we are tracking here: rust-lang/stdarch#738

r? @alexcrichton
@sayantn
Copy link
Contributor

sayantn commented Jun 22, 2024

Intel has fixed this bug (this is not in version 3.6.8 anymore)

@Amanieu Amanieu closed this as completed Jun 29, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@Amanieu @gnzlbg @sayantn