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

ACTS v37.3.0 "-Wmissing-template-arg-list-after-template-kw" issue with clang 19.1.2 #3852

Closed
elmsheus opened this issue Nov 12, 2024 · 1 comment · Fixed by #3846
Closed
Labels
Impact - Major Significant bug and/or affects a lot of modules

Comments

@elmsheus
Copy link

Hi,

I am testing a build of AthenaExternals/Athena with clang 19.1.2 that contains ACTS v37.3.0 and run into the following compilation issue:

[..]
In file included from /build1/elmsheus/clang19_20241112/build/build/AthenaExternals/src/Acts/Core/src/TrackFitting/GlobalChiSquareFitter.cpp:9:
/build1/elmsheus/clang19_20241112/build/build/AthenaExternals/src/Acts/Core/include/Acts/TrackFitting/GlobalChiSquareFitter.hpp:1206:54: error: a template argument li
st is expected after a name prefixed by the template keyword [-Wmissing-template-arg-list-after-template-kw]
 1206 |       auto propagationResult = m_propagator.template propagate(propagatorState);
      |                                                      ^
/build1/elmsheus/clang19_20241112/build/build/AthenaExternals/src/Acts/Core/include/Acts/TrackFitting/GlobalChiSquareFitter.hpp:1209:43: error: a template argument list is expected after a name prefixed by the template keyword [-Wmissing-template-arg-list-after-template-kw]
 1209 |       auto result = m_propagator.template makeResult(std::move(propagatorState),
      |                                           ^
  CXX    dtrace-probe.o
[ 26%] Building CXX object source/processes/CMakeFiles/G4processes-static.dir/biasing/generic/src/G4InteractionLawPhysical.cc.o
/build1/elmsheus/clang19_20241112/build/build/AthenaExternals/src/Acts/Core/include/Acts/TrackFitting/GlobalChiSquareFitter.hpp:1448:54: error: a template argument list is expected after a name prefixed by the template keyword [-Wmissing-template-arg-list-after-template-kw]
 1448 |       auto propagationResult = m_propagator.template propagate(propagatorState);
      |                                                      ^
/build1/elmsheus/clang19_20241112/build/build/AthenaExternals/src/Acts/Core/include/Acts/TrackFitting/GlobalChiSquareFitter.hpp:1451:43: error: a template argument list is expected after a name prefixed by the template keyword [-Wmissing-template-arg-list-after-template-kw]
 1451 |       auto result = m_propagator.template makeResult(std::move(propagatorState),
      |                                           ^
[..]

The -Wmissing-template-arg-list-after-template-kw error is a known new feature of clang 19.1.* and several other external packages run into similar issues.

I've applied the following local patch to get ACTS compiling again, but there might be better code updates:

diff origin/acts-37.3.0/Core/include/Acts/TrackFitting/GlobalChiSquareFitter.hpp acts-37.3.0/Core/include/Acts/TrackFitting/GlobalChiSquareFitter.hpp 
1206c1206
<       auto propagationResult = m_propagator.template propagate(propagatorState);
---
>       auto propagationResult = m_propagator.template propagate<>(propagatorState);
1209c1209
<       auto result = m_propagator.template makeResult(std::move(propagatorState),
---
>       auto result = m_propagator.template makeResult<>(std::move(propagatorState),
1448c1448
<       auto propagationResult = m_propagator.template propagate(propagatorState);
---
>       auto propagationResult = m_propagator.template propagate<>(propagatorState);
1451c1451
<       auto result = m_propagator.template makeResult(std::move(propagatorState),
---
>       auto result = m_propagator.template makeResult<>(std::move(propagatorState),

Hope that's useful.
Cheers, Johannes

@AJPfleger
Copy link
Contributor

Hi Johannes,

thanks for reporting the problem. I think that problem will be solved with:

@AJPfleger AJPfleger linked a pull request Nov 12, 2024 that will close this issue
@AJPfleger AJPfleger added the Impact - Major Significant bug and/or affects a lot of modules label Nov 12, 2024
@kodiakhq kodiakhq bot closed this as completed in #3846 Nov 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Impact - Major Significant bug and/or affects a lot of modules
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants