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

P3107R5 Permit An Efficient Implementation Of <print> #4509

Closed
StephanTLavavej opened this issue Mar 23, 2024 · 1 comment · Fixed by #4821
Closed

P3107R5 Permit An Efficient Implementation Of <print> #4509

StephanTLavavej opened this issue Mar 23, 2024 · 1 comment · Fixed by #4821
Labels
cxx23 C++23 feature defect report Applied retroactively fixed Something works now, yay!

Comments

@StephanTLavavej
Copy link
Member

StephanTLavavej commented Mar 23, 2024

WG21-P3107R5 Permit An Efficient Implementation Of <print>
WG21-P3235R3 std::print More Types Faster With Less Memory

INCREASED feature-test macro:

#define __cpp_lib_print 202406L

These papers were formally accepted as Defect Reports, retroactively amending C++23.

@StephanTLavavej StephanTLavavej added cxx23 C++23 feature defect report Applied retroactively labels Mar 23, 2024
@StephanTLavavej StephanTLavavej moved this from Done to Available in STL C++23 Features Mar 24, 2024
@frederick-vs-ja
Copy link
Contributor

_fputc_nolock and _fwrite_nolock of UCRT seem useful here. However, they still create a __crt_cached_ptd_host guard object for each call, and such overhead seems significant for _fputc_nolock. I wonder whether we can furtherly optimize the underlying calls.

@StephanTLavavej StephanTLavavej moved this from Available to Reviewing PR in STL C++23 Features Jul 8, 2024
@github-project-automation github-project-automation bot moved this from Reviewing PR to Done in STL C++23 Features Sep 4, 2024
@StephanTLavavej StephanTLavavej added the fixed Something works now, yay! label Sep 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cxx23 C++23 feature defect report Applied retroactively fixed Something works now, yay!
Projects
Development

Successfully merging a pull request may close this issue.

2 participants