[go: up one dir, main page]
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.

Sign up for GitHub

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

Jump to bottom

Casting warnings on Windows #715

Open
jhale opened this issue Sep 16, 2024 · 0 comments
Open

Casting warnings on Windows #715

jhale opened this issue Sep 16, 2024 · 0 comments
Assignees
@jhale

Comments

@jhale
Copy link
Member
jhale commented Sep 16, 2024 •
edited
Loading

Originally reported by @minrk.

FEniCS/dolfinx#3198 (comment)

When running the tests on Windows, forgot to mention, this warning came up:

libffcx_forms_1b9958e91fd54e9c55f60346c28378fb63cad26e.c(722): warning C4305: 'initializing': truncation from 'double' to 'const float'

104,127 times when running the tests. I'm not sure if that's something that should be fixed in ffcx.

compiler warnings summary from running the tests with pytest -vs:

# grep -E -o 'warning C.+' dolfinx-win64.txt | sort | uniq -c
1665 warning C4113: 'ufcx_tabulate_tensor_float32 (__cdecl **)' differs in parameter lists from 'void (__cdecl **)(float *,const float *,const float *,const float *,const int *,const uint8_t *)'
1665 warning C4113: 'ufcx_tabulate_tensor_float64 (__cdecl **)' differs in parameter lists from 'void (__cdecl **)(double *,const double *,const double *,const double *,const int *,const uint8_t *)'
 165 warning C4244: 'initializing': conversion from 'double' to 'float', possible loss of data
104127 warning C4305: 'initializing': truncation from 'double' to 'const float'
@jhale jhale self-assigned this Sep 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jhale jhale

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@jhale