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

Double precision not working with -cl-std=c++ #39949

Closed
davidrohr mannequin opened this issue Feb 5, 2019 · 4 comments
Closed

Double precision not working with -cl-std=c++ #39949

davidrohr mannequin opened this issue Feb 5, 2019 · 4 comments
Labels
bugzilla Issues migrated from bugzilla OpenCL

Comments

@davidrohr
Copy link
Mannequin

davidrohr mannequin commented Feb 5, 2019

Bugzilla Link 40603
Resolution FIXED
Resolved on Feb 12, 2019 03:03
Version 8.0
OS Linux
Blocks #39678
CC @AnastasiaStulova,@zmodem

Extended Description

Usage of double precision floating point works with command line argument -cl-std=CL2.0 but not with -cl-std=c++.
Compiling with -cl-std=c++ yields the following error:
error: use of type 'double' requires cl_khr_fp64 extension to be enabled
However, enabling the cl_khr_fp64 extensions yields:
warning: unsupported OpenCL extension 'cl_khr_fp64' - ignoring [-Wignored-pragmas]
For reference: the cl_khr_fp64 extensions works when I set a different -cl-std, e.g. CL2.0

@AnastasiaStulova
Copy link
Contributor

Thanks for reporting this bug!

The fix for this issue is in the review:
https://reviews.llvm.org/D57824

I will commit to master that will end up in the future release of clang 9.0.

Do you need this to be back ported to clang 8.0 as well? If yes, I will have to request this asap.

@davidrohr
Copy link
Mannequin Author

davidrohr mannequin commented Feb 6, 2019

Hi, yes, if you could backport to clang 8, that would be very usefaul.
Thanks.

@AnastasiaStulova
Copy link
Contributor

The fix has been committed in Clang@r353431. This issue now blocks the release 8.0.0, so it should be merged into the release branch soon.

Keeping this open until it's merged into the release branch.

@zmodem
Copy link
Collaborator

zmodem commented Feb 12, 2019

Merged in r353826.

@llvmbot llvmbot transferred this issue from llvm/llvm-bugzilla-archive Dec 10, 2021
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugzilla Issues migrated from bugzilla OpenCL
Projects
None yet
Development

No branches or pull requests

2 participants