-
Notifications
You must be signed in to change notification settings - Fork 407
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
Kokkos_ViewMapping.hpp: intel workaround for using statement #3383
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
My only comment can be addressed in a future PR, no need for it here
c304157
to
3666203
Compare
Apparently failed on format check |
Huh, I ran clang-format locally (clang8). Let me try again... |
Retest this please |
I tried applying clang-format again but did not see a difference, I'm guessing my force-push messed up the CI? s994290:kokkos ndellin$ git log -1
commit 36662031444e10d36f4491f8852205cab15e109c (HEAD -> issue-3381, myfork/issue-3381)
Author: Nathan Ellingwood <ndellin@sandia.gov>
Date: Mon Sep 14 12:57:33 2020 -0600
Kokkos_ViewMapping.hpp: intel workaround for using statement
Temporarily replace using with typedef (and clang-tidy nolint marking)
to workaround intel compiler issue
"error #3186: expected typedef declaration"
Address issue #3381
s994290:kokkos ndellin$ clang-format -i core/src/Kokkos_Macros.hpp core/src/impl/Kokkos_ViewMapping.hpp
s994290:kokkos ndellin$ git status
On branch issue-3381
Your branch is up to date with 'myfork/issue-3381'.
Untracked files:
(use "git add <file>..." to include in what will be committed)
cherry-pick-commits.txt
nothing added to commit but untracked files present (use "git add" to track)
s994290:kokkos ndellin$ clang --version
clang version 8.0.1 (tags/RELEASE_801/final)
Target: x86_64-apple-darwin17.7.0
Thread model: posix
InstalledDir: /opt/local/libexec/llvm-8.0/bin |
core/src/impl/Kokkos_ViewMapping.hpp
Outdated
// typedef work-around for intel compilers error #3186: expected typedef | ||
// declaration | ||
typedef value_type* KOKKOS_IMPL_ALIGN_PTR(KOKKOS_MEMORY_ALIGNMENT) | ||
handle_type; // NOLINT(modernize-use-using) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does this the pass clang-tidy check? We might have to use NOLINTNEXTLINE
or something if not
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just pushed the NOLINTNEXTLINE
change
Temporarily replace using with typedef (and clang-tidy nolint marking) to workaround intel compiler issue "error kokkos#3186: expected typedef declaration" Address issue kokkos#3381
3666203
to
98c05d6
Compare
@dalg24 @masterleinad is there something I need to do differently for the PR to address the CI failures? There are errors like: OMPTARGET-clang
And elsewhere out-of-disk messages during linking attempts:
|
Not your fault. Both failing jobs ran on fetnat05 with Retest this please. |
Temporarily replace using with typedef (and clang-tidy nolint marking)
to workaround intel compiler issue
"error #3186: expected typedef declaration"
Address issue #3381