From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.133.124]) by sourceware.org (Postfix) with ESMTPS id 260D93858C39 for ; Fri, 14 Jan 2022 11:32:43 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org 260D93858C39 Received: from mail-yb1-f200.google.com (mail-yb1-f200.google.com [209.85.219.200]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id us-mta-360-MiNaqTqAOTqjsezxoDEFBw-1; Fri, 14 Jan 2022 06:32:41 -0500 X-MC-Unique: MiNaqTqAOTqjsezxoDEFBw-1 Received: by mail-yb1-f200.google.com with SMTP id v48-20020a25abb3000000b006113ce63ed8so17450910ybi.22 for ; Fri, 14 Jan 2022 03:32:41 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=8HzVc/Z4TRJSZUF8j8Bhd2C+V3N+z0sR2MEoNmkPGdE=; b=w1HBKeeZAIuSj1bonNSfLqNBXUcdbT2DdVCqjWW5tyZxzAISF0f1cnWp0VW0FetvM1 zOlfckI2ZeYkiTr7Hxu1IKGFDkOdpSZZsRMLYOkKt88/XfE/J6KfHpR4WbyWfPk50iyi yRjXy+/ebx2lJjmNZDiARaWkNBWQf8hbJxsh4HTrnlB+VxJ/LmluFuJ7tMrgVopHx0jW JcKJosyVAAL8SIufu3m3UcapF9QbxFJz54il/RdrU09KHQSUKgC1fL8ZJW4Kr9Cw7nRM e72B4/NgmuJbGDksASMyM7wVTZL4BHdB4oJNxgksDEYapdrjQ07TJ3A9ULpHcp1u0oiF kXbw== X-Gm-Message-State: AOAM533zxGrEE5OH5kOfcifGSCmanfjiG2YmKEzgzHpm7T9+1yWkl0kO Fm53Q176xFc0Hm9wU6P+qPvN0hIrPTWOyErSWqjxSEHPADaFUr6MHgyqI1w4Mam9ow5YB35Q9rh lIMhxrZPusK18jlmkjNU+Fksd0lATZ84= X-Received: by 2002:a25:af0b:: with SMTP id a11mr11972569ybh.667.1642159960839; Fri, 14 Jan 2022 03:32:40 -0800 (PST) X-Google-Smtp-Source: ABdhPJwSVtOaxCXS/DfcuNOVtOoMb+Ek+zGS2RKi0FtUey0SFJIpOIdDXzoXqpNOtMbrVMJhpqFcKL8oS2BAQUWDScs= X-Received: by 2002:a25:af0b:: with SMTP id a11mr11972542ybh.667.1642159960551; Fri, 14 Jan 2022 03:32:40 -0800 (PST) MIME-Version: 1.0 References: <20220112093236.2675905-1-jwakely@redhat.com> In-Reply-To: <20220112093236.2675905-1-jwakely@redhat.com> From: Jonathan Wakely Date: Fri, 14 Jan 2022 11:32:29 +0000 Message-ID: Subject: Re: [PATCH] libstdc++: Add attribute to features deprecated in C++17 [PR91260] To: Jonathan Wakely Cc: "libstdc++" , gcc Patches X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com X-Spam-Status: No, score=-7.3 required=5.0 tests=BAYES_00, DKIMWL_WL_HIGH, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, DKIM_VALID_EF, HTML_MESSAGE, RCVD_IN_DNSWL_LOW, RCVD_IN_MSPIKE_H3, RCVD_IN_MSPIKE_WL, SPF_HELO_NONE, SPF_NONE, TXREP autolearn=ham autolearn_force=no version=3.4.4 X-Spam-Checker-Version: SpamAssassin 3.4.4 (2020-01-24) on server2.sourceware.org Content-Type: text/plain; charset="UTF-8" X-Content-Filtered-By: Mailman/MimeDel 2.1.29 X-BeenThere: libstdc++@gcc.gnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Libstdc++ mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 14 Jan 2022 11:32:47 -0000 On Wed, 12 Jan 2022 at 09:34, Jonathan Wakely via Libstdc++ < libstdc++@gcc.gnu.org> wrote: > This passes testing (with -std=gnu++98/11/17/20) but is quite a large > patch for this late in stage 3. Does anybody object to doing this now? > The bugs it fixes were closed as INVALID because we're not actually > *required* to remove these or deprecate them. But users are right to > complain about us silently accepting use of things like std::bind1st in > C++20 mode. Probably WONTFIX would have been better, and in fact we can > fix them (as this patch does). If we don't do this, users will keep > reporting bugs about it, but it could wait for stage 1 if needed. > Pushed to trunk. > === >8 === >8 === >8 === > > There are a lot of things in the C++ standard library which were > deprecated in C++11, and more in C++17. Some of them were removed after > deprecation and are no longer present in the standard at all. We have > not removed these from libstdc++ because keeping them as non-standard > extensions is conforming, and avoids gratuitously breaking user code, > and in some cases we need to keep using them to avoid ABI changes. But > we should at least give a warning for using them. That has not been done > previously because of the library's own uses of them (e.g. the > std::iterator class template used as a base class). > > This adds deprecated attributes to the relevant components, and then > goes through the whole library to add diagnostic pragmas where needed to > suppress warnings about our internal uses of them. The tests are updated > to either expect the additional warnings, or to suppress them where we > aren't interested in them. > > libstdc++-v3/ChangeLog: > > PR libstdc++/91260 > PR libstdc++/91383 > PR libstdc++/95065 > * include/backward/binders.h (bind1st, bind2nd): Add deprecated > attribute. > * include/bits/refwrap.h (_Maybe_unary_or_binary_function): > Disable deprecated warnings for base classes. > (_Reference_wrapper_base): Likewise. > * include/bits/shared_ptr_base.h (_Sp_owner_less): Likewise. > * include/bits/stl_bvector.h (_Bit_iterator_base): Likewise. > * include/bits/stl_function.h (unary_function, binary_function): > Add deprecated attribute. > (unary_negate, not1, binary_negate, not2, ptr_fun) > (pointer_to_unary_function, pointer_to_binary_function) > (mem_fun_t, const_mem_fun_t, mem_fun_ref_t, const_mem_fun_ref_t) > (mem_fun1_t, const_mem_fun1_t, mem_fun_ref1_t) > (const_mem_fun1_ref_t, mem_fun, mem_fun_ref): Add deprecated > attributes. > * include/bits/stl_iterator.h: Disable deprecated warnings for > std::iterator base classes. > * include/bits/stl_iterator_base_types.h (iterator): Add > deprecated attribute. > * include/bits/stl_map.h (map::value_compare): Disable > deprecated warnings for base class. > * include/bits/stl_multimap.h (multimap::value_compare): > Likewise. > * include/bits/stl_raw_storage_iter.h (raw_storage_iterator): > Add deprecated attribute. > * include/bits/stl_tempbuf.h (get_temporary_buffer): Likewise. > * include/bits/stream_iterator.h: Disable deprecated warnings. > * include/bits/streambuf_iterator.h: Likewise. > * include/ext/bitmap_allocator.h: Remove unary_function base > classes. > * include/ext/functional: Disable deprecated warnings. > * include/ext/rope: Likewise. > * include/ext/throw_allocator.h: Likewise. > * include/std/type_traits (result_of): Add deprecated attribute. > * include/tr1/functional: Disable deprecated warnings. > * include/tr1/functional_hash.h: Likewise. > * testsuite/20_util/function_objects/binders/1.cc: Add > -Wno-disable-deprecations. > * testsuite/20_util/function_objects/binders/3113.cc: Likewise. > * testsuite/20_util/function_objects/constexpr.cc: Add > dg-warning. > * testsuite/20_util/raw_storage_iterator/base.cc: Likewise. > * testsuite/20_util/raw_storage_iterator/dr2127.cc: Likewise. > * > testsuite/20_util/raw_storage_iterator/requirements/base_classes.cc: > Likewise. > * > testsuite/20_util/raw_storage_iterator/requirements/explicit_instantiation/1.cc: > Likewise. > * testsuite/20_util/raw_storage_iterator/requirements/typedefs.cc: > Likewise. > * testsuite/20_util/reference_wrapper/24803.cc: > Likewise. > * testsuite/20_util/reference_wrapper/typedefs.cc: Enable for > C++20 and check for absence of nested types. > * testsuite/20_util/shared_ptr/comparison/less.cc: Remove > std::binary_function base class. > * testsuite/20_util/temporary_buffer.cc: Add dg-warning. > * testsuite/21_strings/basic_string/cons/char/69092.cc: Remove > std::iterator base class. > * > testsuite/24_iterators/back_insert_iterator/requirements/base_classes.cc: > Likewise. > * > testsuite/24_iterators/front_insert_iterator/requirements/base_classes.cc: > Likewise. > * > testsuite/24_iterators/insert_iterator/requirements/base_classes.cc: > Likewise. > * > testsuite/24_iterators/istream_iterator/requirements/base_classes.cc: > Likewise. > * testsuite/24_iterators/istreambuf_iterator/92285.cc: > Likewise. > * > testsuite/24_iterators/istreambuf_iterator/requirements/base_classes.cc: > Likewise. > * > testsuite/24_iterators/ostream_iterator/requirements/base_classes.cc: > Likewise. > * > testsuite/24_iterators/ostreambuf_iterator/requirements/base_classes.cc: > Likewise. > * > testsuite/24_iterators/reverse_iterator/requirements/base_classes.cc: > Likewise. > * testsuite/25_algorithms/copy/34595.cc: > Likewise. > * testsuite/25_algorithms/minmax/3.cc: Remove std::binary_function > base class. > * > testsuite/25_algorithms/all_of/requirements/explicit_instantiation/2.cc: > Disable deprecated warnings. > * > testsuite/25_algorithms/all_of/requirements/explicit_instantiation/pod.cc: > Likewise. > * > testsuite/25_algorithms/any_of/requirements/explicit_instantiation/2.cc: > Likewise. > * > testsuite/25_algorithms/any_of/requirements/explicit_instantiation/pod.cc: > Likewise. > * > testsuite/25_algorithms/copy_if/requirements/explicit_instantiation/2.cc: > Likewise. > * > testsuite/25_algorithms/copy_if/requirements/explicit_instantiation/pod.cc: > Likewise. > * > testsuite/25_algorithms/count_if/requirements/explicit_instantiation/2.cc: > Likewise. > * > testsuite/25_algorithms/count_if/requirements/explicit_instantiation/pod.cc: > Likewise. > * > testsuite/25_algorithms/find_end/requirements/explicit_instantiation/2.cc: > Likewise. > * > testsuite/25_algorithms/find_end/requirements/explicit_instantiation/pod.cc: > Likewise. > * > testsuite/25_algorithms/find_first_of/requirements/explicit_instantiation/2.cc: > Likewise. > * > testsuite/25_algorithms/find_first_of/requirements/explicit_instantiation/pod.cc: > Likewise. > * > testsuite/25_algorithms/find_if/requirements/explicit_instantiation/2.cc: > Likewise. > * > testsuite/25_algorithms/find_if/requirements/explicit_instantiation/pod.cc: > Likewise. > * > testsuite/25_algorithms/find_if_not/requirements/explicit_instantiation/2.cc: > Likewise. > * > testsuite/25_algorithms/find_if_not/requirements/explicit_instantiation/pod.cc: > Likewise. > * > testsuite/25_algorithms/for_each/requirements/explicit_instantiation/2.cc: > Likewise. > * > testsuite/25_algorithms/for_each/requirements/explicit_instantiation/pod.cc: > Likewise. > * > testsuite/25_algorithms/is_partitioned/requirements/explicit_instantiation/2.cc: > Likewise. > * > testsuite/25_algorithms/is_partitioned/requirements/explicit_instantiation/pod.cc: > Likewise. > * > testsuite/25_algorithms/is_permutation/requirements/explicit_instantiation/2.cc: > Likewise. > * > testsuite/25_algorithms/is_permutation/requirements/explicit_instantiation/pod.cc: > Likewise. > * > testsuite/25_algorithms/none_of/requirements/explicit_instantiation/2.cc: > Likewise. > * > testsuite/25_algorithms/none_of/requirements/explicit_instantiation/pod.cc: > Likewise. > * > testsuite/25_algorithms/partition/requirements/explicit_instantiation/2.cc: > Likewise. > * > testsuite/25_algorithms/partition/requirements/explicit_instantiation/pod.cc: > Likewise. > * > testsuite/25_algorithms/partition_copy/requirements/explicit_instantiation/2.cc: > Likewise. > * > testsuite/25_algorithms/partition_copy/requirements/explicit_instantiation/pod.cc: > Likewise. > * > testsuite/25_algorithms/partition_point/requirements/explicit_instantiation/2.cc: > Likewise. > * > testsuite/25_algorithms/partition_point/requirements/explicit_instantiation/pod.cc: > Likewise. > * > testsuite/25_algorithms/random_shuffle/requirements/explicit_instantiation/2.cc: > Likewise. > * > testsuite/25_algorithms/random_shuffle/requirements/explicit_instantiation/pod.cc: > Likewise. > * > testsuite/25_algorithms/remove_copy_if/requirements/explicit_instantiation/2.cc: > Likewise. > * > testsuite/25_algorithms/remove_copy_if/requirements/explicit_instantiation/pod.cc: > Likewise. > * > testsuite/25_algorithms/remove_if/requirements/explicit_instantiation/2.cc: > Likewise. > * > testsuite/25_algorithms/remove_if/requirements/explicit_instantiation/pod.cc: > Likewise. > * > testsuite/25_algorithms/replace_copy_if/requirements/explicit_instantiation/2.cc: > Likewise. > * > testsuite/25_algorithms/replace_copy_if/requirements/explicit_instantiation/pod.cc: > Likewise. > * > testsuite/25_algorithms/replace_if/requirements/explicit_instantiation/2.cc: > Likewise. > * > testsuite/25_algorithms/replace_if/requirements/explicit_instantiation/pod.cc: > Likewise. > * > testsuite/25_algorithms/search/requirements/explicit_instantiation/2.cc: > Likewise. > * > testsuite/25_algorithms/search/requirements/explicit_instantiation/pod.cc: > Likewise. > * > testsuite/25_algorithms/search_n/requirements/explicit_instantiation/2.cc: > Likewise. > * > testsuite/25_algorithms/search_n/requirements/explicit_instantiation/pod.cc: > Likewise. > * > testsuite/25_algorithms/stable_partition/requirements/explicit_instantiation/2.cc: > Likewise. > * > testsuite/25_algorithms/stable_partition/requirements/explicit_instantiation/pod.cc: > Likewise. > * > testsuite/25_algorithms/transform/requirements/explicit_instantiation/2.cc: > Likewise. > * > testsuite/25_algorithms/transform/requirements/explicit_instantiation/pod.cc: > Likewise. > * testsuite/27_io/basic_filebuf/underflow/wchar_t/9178.cc: Add > dg-warning. > * testsuite/ext/pb_ds/example/priority_queue_erase_if.cc: > Likewise. > * testsuite/ext/pb_ds/example/priority_queue_split_join.cc: > Likewise. > * testsuite/tr1/3_function_objects/reference_wrapper/typedefs.cc: > Disable deprecated warnings. > * testsuite/tr1/6_containers/hash/requirements/base_classes.cc: > Likewise. > * testsuite/util/regression/trait/erase_if_fn.hpp: Remove > std::unary_function base classes. > * testsuite/util/testsuite_iterators.h (output_iterator_wrapper): > Remove std::iterator base classes. > --- > libstdc++-v3/include/backward/binders.h | 2 + > libstdc++-v3/include/bits/refwrap.h | 11 ++ > libstdc++-v3/include/bits/shared_ptr_base.h | 3 + > libstdc++-v3/include/bits/stl_bvector.h | 4 + > libstdc++-v3/include/bits/stl_function.h | 63 +++++++-- > libstdc++-v3/include/bits/stl_iterator.h | 6 + > .../include/bits/stl_iterator_base_types.h | 2 +- > libstdc++-v3/include/bits/stl_map.h | 3 + > libstdc++-v3/include/bits/stl_multimap.h | 3 + > .../include/bits/stl_raw_storage_iter.h | 7 +- > libstdc++-v3/include/bits/stl_tempbuf.h | 1 + > libstdc++-v3/include/bits/stream_iterator.h | 5 + > .../include/bits/streambuf_iterator.h | 4 + > libstdc++-v3/include/ext/bitmap_allocator.h | 11 +- > libstdc++-v3/include/ext/functional | 5 + > libstdc++-v3/include/ext/rope | 15 +- > libstdc++-v3/include/ext/throw_allocator.h | 5 + > libstdc++-v3/include/std/type_traits | 2 +- > libstdc++-v3/include/tr1/functional | 5 + > libstdc++-v3/include/tr1/functional_hash.h | 5 + > .../20_util/function_objects/binders/1.cc | 1 + > .../20_util/function_objects/binders/3113.cc | 1 + > .../20_util/function_objects/constexpr.cc | 14 +- > .../20_util/raw_storage_iterator/base.cc | 4 +- > .../20_util/raw_storage_iterator/dr2127.cc | 2 +- > .../requirements/base_classes.cc | 4 +- > .../requirements/explicit_instantiation/1.cc | 2 +- > .../requirements/typedefs.cc | 2 +- > .../20_util/reference_wrapper/24803.cc | 4 +- > .../20_util/reference_wrapper/typedefs.cc | 132 ++++++++++++++++-- > .../20_util/shared_ptr/comparison/less.cc | 2 +- > .../testsuite/20_util/temporary_buffer.cc | 2 +- > .../basic_string/cons/char/69092.cc | 9 +- > .../requirements/base_classes.cc | 2 +- > .../requirements/base_classes.cc | 2 +- > .../requirements/base_classes.cc | 2 +- > .../requirements/base_classes.cc | 2 +- > .../24_iterators/istreambuf_iterator/92285.cc | 4 +- > .../requirements/base_classes.cc | 5 +- > .../requirements/base_classes.cc | 2 +- > .../requirements/base_classes.cc | 2 +- > .../requirements/base_classes.cc | 2 +- > .../requirements/explicit_instantiation/2.cc | 3 +- > .../explicit_instantiation/pod.cc | 3 +- > .../requirements/explicit_instantiation/2.cc | 3 +- > .../explicit_instantiation/pod.cc | 3 +- > .../testsuite/25_algorithms/copy/34595.cc | 2 +- > .../requirements/explicit_instantiation/2.cc | 5 +- > .../explicit_instantiation/pod.cc | 5 +- > .../requirements/explicit_instantiation/2.cc | 3 +- > .../explicit_instantiation/pod.cc | 3 +- > .../requirements/explicit_instantiation/2.cc | 3 +- > .../explicit_instantiation/pod.cc | 3 +- > .../requirements/explicit_instantiation/2.cc | 3 +- > .../explicit_instantiation/pod.cc | 3 +- > .../requirements/explicit_instantiation/2.cc | 3 +- > .../explicit_instantiation/pod.cc | 3 +- > .../requirements/explicit_instantiation/2.cc | 3 +- > .../explicit_instantiation/pod.cc | 3 +- > .../requirements/explicit_instantiation/2.cc | 5 +- > .../explicit_instantiation/pod.cc | 5 +- > .../requirements/explicit_instantiation/2.cc | 3 +- > .../explicit_instantiation/pod.cc | 3 +- > .../requirements/explicit_instantiation/2.cc | 1 + > .../explicit_instantiation/pod.cc | 1 + > .../testsuite/25_algorithms/minmax/3.cc | 5 +- > .../requirements/explicit_instantiation/2.cc | 3 +- > .../explicit_instantiation/pod.cc | 3 +- > .../requirements/explicit_instantiation/2.cc | 3 +- > .../explicit_instantiation/pod.cc | 3 +- > .../requirements/explicit_instantiation/2.cc | 3 +- > .../explicit_instantiation/pod.cc | 5 +- > .../requirements/explicit_instantiation/2.cc | 3 +- > .../explicit_instantiation/pod.cc | 3 +- > .../requirements/explicit_instantiation/2.cc | 1 + > .../explicit_instantiation/pod.cc | 1 + > .../requirements/explicit_instantiation/2.cc | 5 +- > .../explicit_instantiation/pod.cc | 5 +- > .../requirements/explicit_instantiation/2.cc | 5 +- > .../explicit_instantiation/pod.cc | 5 +- > .../requirements/explicit_instantiation/2.cc | 5 +- > .../explicit_instantiation/pod.cc | 5 +- > .../requirements/explicit_instantiation/2.cc | 3 +- > .../explicit_instantiation/pod.cc | 3 +- > .../requirements/explicit_instantiation/2.cc | 7 +- > .../explicit_instantiation/pod.cc | 7 +- > .../requirements/explicit_instantiation/2.cc | 7 +- > .../explicit_instantiation/pod.cc | 7 +- > .../requirements/explicit_instantiation/2.cc | 3 +- > .../explicit_instantiation/pod.cc | 3 +- > .../requirements/explicit_instantiation/2.cc | 3 +- > .../explicit_instantiation/pod.cc | 3 +- > .../basic_filebuf/underflow/wchar_t/9178.cc | 2 +- > .../pb_ds/example/priority_queue_erase_if.cc | 2 +- > .../example/priority_queue_split_join.cc | 2 +- > .../reference_wrapper/typedefs.cc | 2 + > .../hash/requirements/base_classes.cc | 1 + > .../util/regression/trait/erase_if_fn.hpp | 8 +- > .../testsuite/util/testsuite_iterators.h | 15 +- > 99 files changed, 439 insertions(+), 138 deletions(-) > > >