[all-commits] [llvm/llvm-project] 6fea34: [libc] Fix non-templated uses of `printf_core::Wri...

[Simon Tatham via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 6fea340023af6ccbaed2d7fe790df565bcf75170
      https://github.com/llvm/llvm-project/commit/6fea340023af6ccbaed2d7fe790df565bcf75170
  Author: Simon Tatham <simon.tatham at arm.com>
  Date:   2025-03-13 (Thu, 13 Mar 2025)

  Changed paths:
    M libc/src/stdio/baremetal/printf.cpp
    M libc/src/stdio/baremetal/vprintf.cpp
    M libc/src/stdio/printf_core/float_dec_converter_limited.h

  Log Message:
  -----------
  [libc] Fix non-templated uses of `printf_core::Writer` (#131149)

Commit 598e882ee88a1e3 turned `Writer` into a template, and updated most
of the call sites that use it. But not all. The alternative FP printf
system in `float_dec_converter_limited.h` wasn't updated, and neither
was `baremetal/printf.cpp` or `baremetal/vprintf.cpp`.

This patch updates `float_dec_converter_limited.h` in the same way that
the previous commit updated `float_dec_converter.h`: propagate the
templatedness through everything in the header, so that anything using a
`Writer` at all has a `write_mode` template parameter to pass on to it.

`printf.cpp` and `vprintf.cpp` are updated in the same way that the
previous commit updated `printf_core/vfprintf_internal.h`: the
`WriteBuffer` has parameter `WriteMode::FLUSH_TO_STREAM`, and `Writer`
picks it up implicitly from that.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications