-
Notifications
You must be signed in to change notification settings - Fork 1
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
DM-45473: Fix aarch64 compile errors and rubin env 9 warning #18
Conversation
This allows tests to pass with small rounding errors, which apparently happen on aarch64.
@@ -5,6 +5,7 @@ project( | |||
license : 'GPL-3.0-or-later', | |||
default_options : ['cpp_std=c++17',], | |||
) | |||
add_project_arguments('-ffp-contract=off', language : 'cpp') |
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.
I don't see this flag propagating to the meson build on aarch64
CFLAGS='-ftree-vectorize -fPIC -fstack-protector-strong -fno-plt -O3 -pipe -isy
stem /lsstsw/miniconda/envs/lsst-scipipe-8.0.0/include'
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.
Also not in any CXXFLAGS
. Doesn't show up in the meson log.
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.
It should show up in build-release/compile_commands.json
. Does it 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.
It does 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.
It does not show up on x86 either with an lsstsw build.
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.
Resolved in that I needed to add it to python/meson.build
since the failures were from numpy image-based templated classes. It may not be necessary here but may as well stay for consistency, or if/when I end up adding GSL image classes.
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.
This fixed the linux_aarch54 build
No description provided.