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

Threads backend doesn't compile with Clang on OS X #1232

Closed
keitat opened this issue Nov 14, 2017 · 6 comments
Closed

Threads backend doesn't compile with Clang on OS X #1232

keitat opened this issue Nov 14, 2017 · 6 comments
Assignees
Labels
Bug Broken / incorrect code; it could be Kokkos' responsibility, or others’ (e.g., Trilinos)
Milestone

Comments

@keitat
Copy link

keitat commented Nov 14, 2017

It will be great if I can get the latest information of Kokkos support for OS X; in particular Clang compilers. The information in README is a bit outdated.

@keitat
Copy link
Author

keitat commented Nov 15, 2017

=== Message from Danny Dunlavy ====
I am using Xcode 8.2:
Apple LLVM version 8.0.0 (clang-800.0.42.1)
Target: x86_64-apple-darwin15.6.0

which does not contain cstdalign.h:

/Users/dmdunla/dev/hpda-tensors/tensor/sparten/tpl/kokkos/core/src/Threads/Kokkos_ThreadsExec.hpp:53:10: fatal error: 'cstdalign' file not found
#include

I looked online and it is not clear whether Xcode 9 has it either. So this dependency means that Apple clang may not be an option for building.

@crtrott crtrott added Bug Broken / incorrect code; it could be Kokkos' responsibility, or others’ (e.g., Trilinos) Blocks Promotion Overview issue for release-blocking bugs labels Nov 18, 2017
@ibaned ibaned changed the title Kokko's Clang Support on OS X Threads backend <cstdalign> not found by OS X Clang Nov 23, 2017
@ibaned ibaned changed the title Threads backend <cstdalign> not found by OS X Clang Threads backend doesn't compile with Clang on OS X Nov 23, 2017
@ibaned
Copy link
Contributor

ibaned commented Nov 23, 2017

Can confirm this is still an issue with:

Apple LLVM version 9.0.0 (clang-900.0.38)

@ibaned ibaned self-assigned this Nov 23, 2017
@ibaned
Copy link
Contributor

ibaned commented Nov 23, 2017

It is not an issue with:

clang version 5.0.0-svn301099-1~exp1

on Ubuntu.
However, on both the OS X and Ubuntu platforms I have, Kokkos Core and its tests still compile with <cstdalign> removed, under Clang, suggesting the header may not be needed.

@ibaned
Copy link
Contributor

ibaned commented Nov 23, 2017

The include was added in ead19d5 and only has to do with alignas, but grep says that kokkos/core/ contains no references to alignas, we instead use __attribute__((aligned(N))) for alignment control. This further suggests it can be removed.

@ibaned
Copy link
Contributor

ibaned commented Nov 23, 2017

Pull request #1242 should fix this

@ibaned
Copy link
Contributor

ibaned commented Nov 23, 2017

@keitat in short, Kokkos needs to be able to build with the latest Clang on OS X, it doesn't, and we're fixing that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Broken / incorrect code; it could be Kokkos' responsibility, or others’ (e.g., Trilinos)
Projects
None yet
Development

No branches or pull requests

4 participants