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

ambiguous overload of count_digits() on xtensa esp32s3 gcc 12.2.0 (20230208) #4109

Closed
m-mcgowan opened this issue Aug 5, 2024 · 2 comments
Closed

Comments

@m-mcgowan
Copy link

godbolt repro

In file included from /opt/compiler-explorer/libs/fmt/trunk/include/fmt/core.h:5,
                 from <source>:1:
/opt/compiler-explorer/libs/fmt/trunk/include/fmt/format.h: In function 'constexpr OutputIt fmt::v11::detail::write_exponent(int, OutputIt)':
/opt/compiler-explorer/libs/fmt/trunk/include/fmt/format.h:1608:56: error: call of overloaded 'count_digits(unsigned int&)' is ambiguous
 1608 |     return format_decimal<Char>(out, uexp, count_digits(uexp));
      |                                            ~~~~~~~~~~~~^~~~~~
/opt/compiler-explorer/libs/fmt/trunk/include/fmt/format.h:1183:29: note: candidate: 'int fmt::v11::detail::count_digits(uint64_t)'
 1183 | FMT_CONSTEXPR20 inline auto count_digits(uint64_t n) -> int {
      |                             ^~~~~~~~~~~~
/opt/compiler-explorer/libs/fmt/trunk/include/fmt/format.h:1233:29: note: candidate: 'int fmt::v11::detail::count_digits(uint32_t)'
 1233 | FMT_CONSTEXPR20 inline auto count_digits(uint32_t n) -> int {
      |                             ^~~~~~~~~~~~
Compiler returned: 1
@vitaut
Copy link
Contributor

vitaut commented Aug 9, 2024

Fixed in 9831431. Thanks for reporting!

@vitaut vitaut closed this as completed Aug 9, 2024
@m-mcgowan
Copy link
Author

Thank you for the fix! I looked at the code, but couldn't grasp it well enough to suggest a fix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants