Changes in streambuf

2ae52326 - [libc++] Towards a simpler extern template story in libc++

Tue, 08 Jun 2021 21:25:08 +0000

[libc++] Towards a simpler extern template story in libc++The flexibility around extern template instantiation declarations inlibc++ result in a very complicated model, especially when support forslightly different configurations (like the debug mode or assertionsin the dylib) are taken into account. That results in unexpected bugslike http://llvm.org/PR50534 (and there have been multiple similarbugs in the past, notably around the debug mode).This patch gets rid of the _LIBCPP_DISABLE_EXTERN_TEMPLATE knob, whichI don't think is fundamental. Indeed, the motivation for that knob was toavoid taking a dependency on the library, however that can be done betterby linking against the static library instead. And in fact, some parts ofthe headers will always depend on things defined in the library, whichdefeats the original goal of _LIBCPP_DISABLE_EXTERN_TEMPLATE.Differential Revision: https://reviews.llvm.org/D103960 List of files: /llvm-project-15.0.7/libcxx/include/streambuf

385cc25a - [libc++] Ensure that all public C++ headers include <__assert>

Fri, 25 Mar 2022 16:55:36 +0000

[libc++] Ensure that all public C++ headers include <__assert>This patch changes the requirement for getting the declaration of theassertion handler from including <__assert> to including any publicC++ header of the library. Note that C compatibility headers areexcluded because we don't implement all the C headers ourselves --some of them are taken straight from the C library, like assert.h.It also adds a generated test to check it. Furthermore, this newgenerated test is designed in a way that will make it possible toreplace almost all the existing test-generation scripts with thissystem in upcoming patches.Differential Revision: https://reviews.llvm.org/D122506 List of files: /llvm-project-15.0.7/libcxx/include/streambuf

fa6b9e40 - [libc++] Normalize all our '#pragma GCC system_header', and regression-test.

Wed, 02 Feb 2022 01:16:40 +0000

[libc++] Normalize all our '#pragma GCC system_header', and regression-test.Now we'll notice if a header forgets to include this magic phrase.Differential Revision: https://reviews.llvm.org/D118800 List of files: /llvm-project-15.0.7/libcxx/include/streambuf

5726e559 - [libc++] Modularize

Mon, 10 Jan 2022 19:56:43 +0000

[libc++] Modularize I didn't split the calendar bits more than this because there was littlebenefit to doing it, and I know our calendar support is incomplete.Whoever picks up the missing calendar bits can organize these headersat their leisure.Differential Revision: https://reviews.llvm.org/D116965 List of files: /llvm-project-15.0.7/libcxx/include/streambuf

bd6e6846 - [libc++] Add the version header to all headers.

Wed, 22 Dec 2021 17:14:14 +0000

[libc++] Add the version header to all headers.Some headers which require the version header depend on other headers toprovide it. Include the version header in all top-level headers to makesure a header cleanup can't remove the version header.Note this doesn't add the version header to the c headers.Reviewed By: #libc, Quuxplusone, ldionneDifferential Revision: https://reviews.llvm.org/D116172 List of files: /llvm-project-15.0.7/libcxx/include/streambuf

eb8650a7 - [runtimes][NFC] Remove filenames at the top of the license notice

Wed, 17 Nov 2021 21:25:01 +0000

[runtimes][NFC] Remove filenames at the top of the license noticeWe've stopped doing it in libc++ for a while now because these nameswould end up rotting as we move things around and copy/paste stuff.This cleans up all the existing files so as to stop the spreadingas people copy-paste headers around. List of files: /llvm-project-15.0.7/libcxx/include/streambuf

bfbd73f8 - [libc++] Alphabetize and include-what-you-use. NFCI.

Wed, 19 May 2021 15:57:04 +0000

[libc++] Alphabetize and include-what-you-use. NFCI.Differential Revision: https://reviews.llvm.org/D102781 List of files: /llvm-project-15.0.7/libcxx/include/streambuf

4cd6ca10 - [libc++] NFC: Normalize `#endif //` comment indentation

Tue, 20 Apr 2021 16:03:32 +0000

[libc++] NFC: Normalize `#endif //` comment indentation List of files: /llvm-project-15.0.7/libcxx/include/streambuf

27e5bffb - [libc++] Fix the header guard from _LIBCPP_STEAMBUF to _LIBCPP_STREAMBUF.

Sun, 04 Apr 2021 21:39:50 +0000

[libc++] Fix the header guard from _LIBCPP_STEAMBUF to _LIBCPP_STREAMBUF. List of files: /llvm-project-15.0.7/libcxx/include/streambuf

527a7fdf - [libc++] Replace several uses of 0 by nullptr

Tue, 24 Nov 2020 17:53:53 +0000

[libc++] Replace several uses of 0 by nullptrDifferential Revision: https://reviews.llvm.org/D43159 List of files: /llvm-project-15.0.7/libcxx/include/streambuf

ce565861 - [libc++] Drop old workaround for iostreams instantiations missing from the dylib

Thu, 22 Oct 2020 18:43:41 +0000

[libc++] Drop old workaround for iostreams instantiations missing from the dylibOn old Apple platforms (pre 10.9), we couldn't rely on the iostreamsexplicit instantiations being part of the dylib. However, we don'tsupport back-deploying to such old deployment targets anymore, so theworkaround can be dropped. List of files: /llvm-project-15.0.7/libcxx/include/streambuf

57b08b09 - Update more file headers across all of the LLVM projects in the monorepo

Sat, 19 Jan 2019 10:56:40 +0000

Update more file headers across all of the LLVM projects in the monorepoto reflect the new license. These used slightly different spellings thatdefeated my regular expressions.We understand that people may be surprised that we're moving the headerentirely to discuss the new license. We checked this carefully with theFoundation's lawyer and we believe this is the correct approach.Essentially, all code in the project is now made available by the LLVMproject under our new license, so you will see that the license headersinclude that license only. Some of our contributors have contributedcode under our old license, and accordingly, we have retained a copy ofour old license notice in the top-level files in each project andrepository.llvm-svn: 351648 List of files: /llvm-project-15.0.7/libcxx/include/streambuf

f7f5a1f7 - [libcxx] Don't depend on availability markup to provide the streams in the dylib

Thu, 06 Dec 2018 00:24:58 +0000

[libcxx] Don't depend on availability markup to provide the streams in the dylibWhether an explicit instantiation declaration should be provided is nota matter of availability markup.This problem is exemplified by the fact that some tests were incorrectlymarked as XFAIL when they should instead have been using the definitionof streams from the headers, and hence passing, and that, regardless ofwhether visibility annotations are enabled.llvm-svn: 348436 List of files: /llvm-project-15.0.7/libcxx/include/streambuf

7833b88a - [libcxx] Remove _LIBCPP_EXTERN_TEMPLATE_INLINE_VISIBILITY

Tue, 16 Oct 2018 19:26:23 +0000

[libcxx] Remove _LIBCPP_EXTERN_TEMPLATE_INLINE_VISIBILITYThat macro has been defined to _LIBCPP_HIDE_FROM_ABI_AFTER_V1 for manyweeks now, so we're actually replacing uses of it for uses of_LIBCPP_HIDE_FROM_ABI_AFTER_V1 directly.This should not change or break anything since the two macros are100% equivalent, unless somebody is (incorrectly!) relying on_LIBCPP_EXTERN_TEMPLATE_INLINE_VISIBILITY being defined.llvm-svn: 344641 List of files: /llvm-project-15.0.7/libcxx/include/streambuf

dc7200b4 - [libc++] Take 2: Replace uses of _LIBCPP_ALWAYS_INLINE by _LIBCPP_INLINE_VISIBILITY

Wed, 11 Jul 2018 23:14:33 +0000

[libc++] Take 2: Replace uses of _LIBCPP_ALWAYS_INLINE by _LIBCPP_INLINE_VISIBILITYSummary:We never actually mean to always inline a function -- all the uses ofthe macro I could find are actually attempts to control the visibilityof symbols. This is better described by _LIBCPP_INLINE_VISIBILITY, whichis actually always defined the same.This change is orthogonal to the decision of what we're actually goingto do with _LIBCPP_INLINE_VISIBILITY -- it just simplifies things byhaving one canonical way of doing things.Note that this commit had originally been applied in r336369 and thenreverted in r336382 because of unforeseen problems. Both of these problemshave now been fixed.Reviewers: EricWF, mclow.listsSubscribers: christof, dexonsmith, erikvanderpoelDifferential Revision: https://reviews.llvm.org/D48892llvm-svn: 336866 List of files: /llvm-project-15.0.7/libcxx/include/streambuf

195a499d - Revert "[libc++] Replace uses of _LIBCPP_ALWAYS_INLINE by _LIBCPP_INLINE_VISIBILITY"

Thu, 05 Jul 2018 18:41:50 +0000

Revert "[libc++] Replace uses of _LIBCPP_ALWAYS_INLINE by _LIBCPP_INLINE_VISIBILITY"This reverts commit r336369. The commit had two problems:1. __pbump was marked as _LIBCPP_EXTERN_TEMPLATE_INLINE_VISIBILITY instead of _LIBCPP_INLINE_VISIBILITY, which lead to two symbols being added in the dylib and the check-cxx-abilist failing.2. The LLDB tests started failing because they undefine `_LIBCPP_INLINE_VISIBILITY`. I need to figure out why they do that and fix the tests before we can go forward with this change.llvm-svn: 336382 List of files: /llvm-project-15.0.7/libcxx/include/streambuf

4a8f3f99 - [libc++] Replace uses of _LIBCPP_ALWAYS_INLINE by _LIBCPP_INLINE_VISIBILITY

Thu, 05 Jul 2018 16:49:38 +0000

[libc++] Replace uses of _LIBCPP_ALWAYS_INLINE by _LIBCPP_INLINE_VISIBILITYSummary:We never actually mean to always inline a function -- all the uses ofthe macro I could find are actually attempts to control the visibilityof symbols. This is better described by _LIBCPP_INLINE_VISIBILITY, whichis actually always defined the same.This change is orthogonal to the decision of what we're actually goingto do with _LIBCPP_INLINE_VISIBILITY -- it just simplifies things byhaving one canonical way of doing things.Reviewers: EricWFSubscribers: christof, llvm-commits, dexonsmith, erikvanderpoel, mclow.listsDifferential Revision: https://reviews.llvm.org/D48892llvm-svn: 336369 List of files: /llvm-project-15.0.7/libcxx/include/streambuf

1a6493b4 - Add static_asserts to basic_ios and basic_stream_buf to ensure that that the traits match the character type. This is a requirement on the user - now we get consistent failures at compile time instead of incomprehensible error messages or runtime failures. This is also LWG#2994 - not yet adopted.

Thu, 01 Feb 2018 03:55:27 +0000

Add static_asserts to basic_ios and basic_stream_buf to ensure that that the traits match the character type. This is a requirement on the user - now we get consistent failures at compile time instead of incomprehensible error messages or runtime failures. This is also LWG#2994 - not yet adopted.llvm-svn: 323945 List of files: /llvm-project-15.0.7/libcxx/include/streambuf

d90758e2 - Make pbump (internally) handle sizes bigger than MAX_INT. Fixes PR#33725 - thanks to Jonathan Wakely for the report

Tue, 12 Sep 2017 15:00:43 +0000

Make pbump (internally) handle sizes bigger than MAX_INT. Fixes PR#33725 - thanks to Jonathan Wakely for the reportllvm-svn: 313031 List of files: /llvm-project-15.0.7/libcxx/include/streambuf

e5cc2fcc - make sure that we don't call basic_streambuf::gbump with a value bigger than INT_MAX, since it only takes an int. Related to, but not quite the same as PR33725

Mon, 24 Jul 2017 14:05:10 +0000

make sure that we don't call basic_streambuf::gbump with a value bigger than INT_MAX, since it only takes an int. Related to, but not quite the same as PR33725llvm-svn: 308880 List of files: /llvm-project-15.0.7/libcxx/include/streambuf