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

Tpetra: missing impl_scalar_t in recent changes in block Crs matrix helpers #12235

Closed
maartenarnst opened this issue Sep 11, 2023 · 2 comments
Closed
Labels
type: bug The primary issue is a bug in Trilinos code or tests

Comments

@maartenarnst
Copy link
Contributor

@jgfouca @csiefer2

@trilinos/tpetra

Bug Report

We are compiling Trilinos with the complex double type enabled. We are seeing a compilation error when the file Tpetra_BlockCrsMatrix_Helpers_def.hpp is compiled:

INFO:root:#21 1363.9 .../packages/tpetra/core/src/Tpetra_BlockCrsMatrix_Helpers_def.hpp:574:22: error: reference to __host__ function 'operator complex' in __host__ __device__ function
INFO:root:#21 1363.9         const Scalar value = values(row_nnz);

I think I tracked it down to this commit:

which moves some code into Tpetra_BlockCrsMatrix_Helpers_decl.hpp from shylu but doesn't seem to take into account that there's the implementation scalar type in Tpetra.

I'll introduce a PR with a fix.

@csiefer2
Copy link
Member

@maartenarnst That looks reasonable. Thanks for the fix!

@maartenarnst
Copy link
Contributor Author

maartenarnst commented Sep 15, 2023

Closing this because #12236 was merged.

@jhux2 jhux2 added this to Tpetra Aug 12, 2024
@jhux2 jhux2 moved this to Done in Tpetra Aug 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug The primary issue is a bug in Trilinos code or tests
Projects
Status: Done
Development

No branches or pull requests

2 participants