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

SEACAS: build error under Intel 2020.2 #10172

Closed
jhux2 opened this issue Feb 8, 2022 · 9 comments
Closed

SEACAS: build error under Intel 2020.2 #10172

jhux2 opened this issue Feb 8, 2022 · 9 comments
Labels
client: ExaWind All issue that impact the ECP project ExaWind pkg: seacas type: bug The primary issue is a bug in Trilinos code or tests

Comments

@jhux2
Copy link
Member

jhux2 commented Feb 8, 2022

Bug Report

@trilinos/seacas @gsjaardema @jrood-nrel @PaulMullowney

Description

Colleagues have reported the following Trilinos build error with Intel 2020.2 and (possibly) with gcc 9.3.0. This is for Trilinos develop ef73d14babf. (A previous build with cbcdc150777 succeeded.

  >> 3952    /projects/ecp/exawind/exawind-testing/spack-manager/stage/jrood/sp
             ack-stage-trilinos-develop-srdtfhnbtdkqj7o2jp2auisdi4vlpqlr/spack-
             src/packages/seacas/libraries/ioss/src/private_copy_fmt/fmt/core.h
             (1706): error: static assertion failed with "Cannot format a const
              argument."
     3953        static_assert(formattable_const, "Cannot format a const argume
             nt.");
     3954        ^
     3955              detected during:
     3956                instantiation of "auto fmt::v8::detail::make_arg<IS_PA
             CKED,Context,<unnamed>,T,<unnamed>>(T &&)->fmt::v8::detail::value<
             Context> [with IS_PACKED=true, Context=fmt::v8::basic_format_conte
             xt<fmt::v8::detail::buffer_appender<char>, char>, <unnamed>=fmt::v
             8::detail::type::custom_type, T=const fmt::v8::remove_cvref_t<cons
             t fmt::v8::remove_reference_t<fmt::v8::group_digits_view<std::size
             _t={unsigned long}>> &> &, <unnamed>=0]" at line 1848
@jhux2 jhux2 added type: bug The primary issue is a bug in Trilinos code or tests pkg: seacas client: ExaWind All issue that impact the ECP project ExaWind labels Feb 8, 2022
@jhux2
Copy link
Member Author

jhux2 commented Feb 10, 2022

I've verified that by reverting e2a591dca, I can get past this error.

@gsjaardema
Copy link
Contributor

I am working with the lib::fmt developers to see if we can get a fix for the intel compiler. Hopefully something today, but maybe better to just revert...

@jrood-nrel
Copy link

Can we get this reverted?

@jhux2
Copy link
Member Author

jhux2 commented Feb 14, 2022

@jrood-nrel PR #10179 is in flight to fix this.

@jrood-nrel
Copy link

I think it would be best to revert this so our GPU tests can run again.

@jhux2
Copy link
Member Author

jhux2 commented Feb 15, 2022

@jrood-nrel I think that's the plan if @gsjaardema can't resolve this today.

@jhux2
Copy link
Member Author

jhux2 commented Feb 16, 2022

@jrood-nrel #10179 should fix this issue. (Thank you, @gsjaardema!) I verified that my build on Summit finishes successfully with the PR, and fails without it. Once you've verified that your builds are clean, could you please close?

@jrood-nrel
Copy link

Sure. Thanks!

@jrood-nrel
Copy link

This looks fixed, but I can't close it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
client: ExaWind All issue that impact the ECP project ExaWind pkg: seacas type: bug The primary issue is a bug in Trilinos code or tests
Projects
None yet
Development

No branches or pull requests

3 participants