[{"id":3680377,"web_url":"http://patchwork.ozlabs.org/comment/3680377/","msgid":"<CACb0b4m4vFBaF6oBN2Tj3Cp0kRTfoTMKRyedMJqb0PUY9=f-ng@mail.gmail.com>","list_archive_url":null,"date":"2026-04-22T09:28:43","subject":"Re: [PATCH] libstdc++: Accept data_handle_type by value in mdspan\n deduction guide.","submitter":{"id":48004,"url":"http://patchwork.ozlabs.org/api/people/48004/","name":"Jonathan Wakely","email":"jwakely@redhat.com"},"content":"On Wed, 22 Apr 2026 at 08:35, Tomasz Kamiński <tkaminsk@redhat.com> wrote:\n>\n> This makes the deduction guide accepting data_handle_type, mapping\n> and accessor consistent with corresponding constructor.\n> Resolves LWG4511.\n>\n> libstdc++-v3/ChangeLog:\n>\n>         * include/std/mdspan (mdspan): Remove reference from\n>         _AccessorType::data_handle_type parameteter of deduction\n>         guide.\n>         * testsuite/23_containers/mdspan/mdspan.cc: New test.\n> ---\n> Testing on x86_64-linux locally. All *mdspan* test passed in C++23,C++26\n> mode and with modules. OK for trunk when all test passes?\n\nOK, thanks\n\n\n>\n>  libstdc++-v3/include/std/mdspan               |  4 +++-\n>  .../testsuite/23_containers/mdspan/mdspan.cc  | 19 +++++++++++++++++++\n>  2 files changed, 22 insertions(+), 1 deletion(-)\n>\n> diff --git a/libstdc++-v3/include/std/mdspan b/libstdc++-v3/include/std/mdspan\n> index f30c5e83d16..fae4ce0baa2 100644\n> --- a/libstdc++-v3/include/std/mdspan\n> +++ b/libstdc++-v3/include/std/mdspan\n> @@ -3267,8 +3267,10 @@ _GLIBCXX_BEGIN_NAMESPACE_VERSION\n>      -> mdspan<_ElementType, typename _MappingType::extents_type,\n>               typename _MappingType::layout_type>;\n>\n> +  // _GLIBCXX_RESOLVE_LIB_DEFECTS\n> +  // 4511. Inconsistency between the deduction guide of std::mdspan taking...\n>    template<typename _MappingType, typename _AccessorType>\n> -    mdspan(const typename _AccessorType::data_handle_type&, const _MappingType&,\n> +    mdspan(typename _AccessorType::data_handle_type, const _MappingType&,\n>            const _AccessorType&)\n>      -> mdspan<typename _AccessorType::element_type,\n>               typename _MappingType::extents_type,\n> diff --git a/libstdc++-v3/testsuite/23_containers/mdspan/mdspan.cc b/libstdc++-v3/testsuite/23_containers/mdspan/mdspan.cc\n> index 270b3f18daa..62ff7201cce 100644\n> --- a/libstdc++-v3/testsuite/23_containers/mdspan/mdspan.cc\n> +++ b/libstdc++-v3/testsuite/23_containers/mdspan/mdspan.cc\n> @@ -404,6 +404,24 @@ template<typename CustomInt, bool ValidForPacks, bool ValidForArrays>\n>      return true;\n>    }\n>\n> +void\n> +test_from_volatile_ptr_and_accessor()\n> +{\n> +  constexpr size_t n = 3*5*7;\n> +  std::array<double, n> storage{};\n> +  double* volatile vptr = storage.data();\n> +  using Extents = std::extents<int, 3, 5, 7>;\n> +\n> +  auto exts = Extents{};\n> +  auto m = std::layout_left::mapping(exts);\n> +  auto a = std::default_accessor<double>{};\n> +  auto md = std::mdspan(vptr, m, a);\n> +\n> +  assert_deduced_typedefs<double, Extents, std::layout_left>(md);\n> +  VERIFY(md.data_handle() == storage.data());\n> +  VERIFY(md.mapping() == m);\n> +}\n> +\n>  template<typename T, bool NothrowConstructible = true,\n>          bool NothrowAssignable = true>\n>    class OpaqueAccessor\n> @@ -757,6 +775,7 @@ main()\n>    test_from_int_like<RValueInt, true, false>();\n>    test_from_int_like<ConstLValueInt, false, true>();\n>\n> +  test_from_volatile_ptr_and_accessor();\n>    test_from_opaque_accessor();\n>    test_from_base_class_accessor();\n>    test_from_mapping_like();\n> --\n> 2.53.0\n>","headers":{"Return-Path":"<gcc-patches-bounces~incoming=patchwork.ozlabs.org@gcc.gnu.org>","X-Original-To":["incoming@patchwork.ozlabs.org","gcc-patches@gcc.gnu.org"],"Delivered-To":["patchwork-incoming@legolas.ozlabs.org","gcc-patches@gcc.gnu.org"],"Authentication-Results":["legolas.ozlabs.org;\n\tdkim=pass (1024-bit key;\n unprotected) header.d=redhat.com header.i=@redhat.com header.a=rsa-sha256\n header.s=mimecast20190719 header.b=SKCb7oxP;\n\tdkim-atps=neutral","legolas.ozlabs.org;\n spf=pass (sender SPF authorized) smtp.mailfrom=gcc.gnu.org\n (client-ip=2620:52:6:3111::32; helo=vm01.sourceware.org;\n envelope-from=gcc-patches-bounces~incoming=patchwork.ozlabs.org@gcc.gnu.org;\n receiver=patchwork.ozlabs.org)","sourceware.org;\n\tdkim=pass (1024-bit key,\n unprotected) header.d=redhat.com header.i=@redhat.com header.a=rsa-sha256\n header.s=mimecast20190719 header.b=SKCb7oxP","sourceware.org; dmarc=pass (p=quarantine dis=none)\n header.from=redhat.com","sourceware.org; spf=pass smtp.mailfrom=redhat.com","server2.sourceware.org;\n arc=none smtp.remote-ip=170.10.133.124"],"Received":["from vm01.sourceware.org (vm01.sourceware.org\n [IPv6:2620:52:6:3111::32])\n\t(using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)\n\t key-exchange x25519 server-signature ECDSA (secp384r1) server-digest SHA384)\n\t(No client certificate requested)\n\tby legolas.ozlabs.org (Postfix) with ESMTPS id 4g0v8L6dVPz1yCv\n\tfor <incoming@patchwork.ozlabs.org>; Wed, 22 Apr 2026 19:30:09 +1000 (AEST)","from vm01.sourceware.org (localhost [127.0.0.1])\n\tby sourceware.org (Postfix) with ESMTP id 611964BAD150\n\tfor <incoming@patchwork.ozlabs.org>; Wed, 22 Apr 2026 09:30:07 +0000 (GMT)","from us-smtp-delivery-124.mimecast.com\n (us-smtp-delivery-124.mimecast.com [170.10.133.124])\n by sourceware.org (Postfix) with ESMTP id 391054BAD158\n for <gcc-patches@gcc.gnu.org>; Wed, 22 Apr 2026 09:29:02 +0000 (GMT)","from mail-yx1-f71.google.com (mail-yx1-f71.google.com\n [74.125.224.71]) by relay.mimecast.com with ESMTP with STARTTLS\n (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id\n us-mta-524-ejFUNlB-NquEQuLKZJfXBg-1; Wed, 22 Apr 2026 05:29:00 -0400","by mail-yx1-f71.google.com with SMTP id\n 956f58d0204a3-6545e0d6506so96440d50.0\n for <gcc-patches@gcc.gnu.org>; Wed, 22 Apr 2026 02:29:00 -0700 (PDT)"],"DKIM-Filter":["OpenDKIM Filter v2.11.0 sourceware.org 611964BAD150","OpenDKIM Filter v2.11.0 sourceware.org 391054BAD158"],"DMARC-Filter":"OpenDMARC Filter v1.4.2 sourceware.org 391054BAD158","ARC-Filter":"OpenARC Filter v1.0.0 sourceware.org 391054BAD158","ARC-Seal":"i=1; a=rsa-sha256; d=sourceware.org; s=key; t=1776850142; cv=none;\n b=qkYCBIbn4wKd/OincCv7tbJ1XoxxAj8QTYtocgpbuuJe/2upZQKR9gwKwJAliUIih6PTFdTvqAkmsMxXmrf82CzIQeeyrHQIZu2P6VX+BrjU3Ik/ilJ57EVpOLkEcFQMQq6NZ7ieg4Aj5hfMplwgSUeNEiUvNLnpbltyjRoQC18=","ARC-Message-Signature":"i=1; a=rsa-sha256; d=sourceware.org; s=key;\n t=1776850142; c=relaxed/simple;\n bh=Xp8h8Lq7qea6BqxxByy/HWA/vqUUyzW68kYc1f59Kp8=;\n h=DKIM-Signature:MIME-Version:From:Date:Message-ID:Subject:To;\n b=EkAsaZnrMxnqkf5n8QczgUG62DQhdjumwDNyo/H+E8BuY+Wj8J29BqRORgSlwavqrgK9CUdqlDBVqNT3dICjEl7gIp8uXgPsO3TOsa1yPJmvzAlRNU7DlDW2PAqBTsbyZbNPkD5G3eIOopDB13UsV9YVvJqXqoPak8HohfLet5A=","ARC-Authentication-Results":"i=1; server2.sourceware.org","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com;\n s=mimecast20190719; t=1776850141;\n h=from:from:reply-to:subject:subject:date:date:message-id:message-id:\n to:to:cc:cc:mime-version:mime-version:content-type:content-type:\n content-transfer-encoding:content-transfer-encoding:\n in-reply-to:in-reply-to:references:references;\n bh=AKd6Yg4f9/p99UvC+P2ak0naJiEOfVVNrXokYbihSxQ=;\n b=SKCb7oxPDIbaBqFz/zCxLBForXKuW2NyTnEWzR4BdKN23vSd6YcM7DdEyjRLEA86iy85Bq\n sFlA1388i3tvsi/gwSj8Wng+03Ce+Sp0eDj4O2Nh22blA+9BFSgyF1BbgvamUN7b+x7ks2\n 7Ted+rz5Pap5QPEa7Ik2yzRdEX5xJ78=","X-MC-Unique":"ejFUNlB-NquEQuLKZJfXBg-1","X-Mimecast-MFC-AGG-ID":"ejFUNlB-NquEQuLKZJfXBg_1776850140","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n d=1e100.net; s=20251104; t=1776850140; x=1777454940;\n h=content-transfer-encoding:cc:to:subject:message-id:date:from\n :in-reply-to:references:mime-version:x-gm-gg:x-gm-message-state:from\n :to:cc:subject:date:message-id:reply-to;\n bh=AKd6Yg4f9/p99UvC+P2ak0naJiEOfVVNrXokYbihSxQ=;\n b=VteRXRmHmBHjW0HleM6YMSAzO43jzLl1YCFcRWT0kbzzMHjfhtvILUstbxVQP8msJp\n bes2nxlL2R9GG4EsZaKvJ3nAriXQ8vyn7ORY6sQDPEcRvND3DyOpW/ExcZ8bihZ5a+YC\n 4ctAO5SfwimFe58QZTSJXd2gsUagowSVl3kdPTQxftSfc8Oq4zmS9PWFMwcJX18W+l0T\n aYHGYDuI0fluetGeSrCXHnnLQgim+TGdrPCMHmtXcGXKQjDbKXkqbqeXQwjewQyffc+3\n izWtKaM4WCz457k7JDdY/38USf8t1BK0cYaPNsPnNhqaYzxWZSZKhu6kX5zKI/4PEid0\n zWBQ==","X-Forwarded-Encrypted":"i=1;\n AFNElJ/hDl14uGMm7Us8m3JlGs9/4dWSkeNHj6u/L52LDTlJyWWEmVG4Bu6aX/NCTPI+YrKDzY/G39odvBUapQ==@gcc.gnu.org","X-Gm-Message-State":"AOJu0YxpIN/9sFoneflN9jFVe9ghqez35+g1aFIiJxsVnZcd8OkkfZ00\n shW4UmdkDVrMLZc4U4rJqSFy+bTrTBdNKbch5TtpMm9nsXwoGTPzJrmOHYxlRV9cIM+yGmQ1w7U\n wVj5Rh3P01FgNx2VKV2zBM65tR/ETrYzZtXgH/43knetuU0DomFNnOY52ygDrGQCV35LqTt+IQz\n djXvjKWahY0Uyd8HCASBlilIZoUP+FQ/5oyA==","X-Gm-Gg":"AeBDieveOuEjmP1GVhv60KfStw4c/IDgiFZSmG/S0xfrR7oPjhTtKTXdVrCMUJLVJIu\n jvKrFgENo9J5drjwnPOJmbpdw8vt7IyapSYiItH2oP3GkWwt6OlgiwXM/7BdahribJjXkOfCeIO\n EU+DS2S6RfyQ0ds64FP/dreUK6NLW12ImAAyyjCSTa13yHKDJ+el6Ut592hPv4xsIBgp2Cbn3fB\n MhoxTgnuJoadUN73EYNiyp//vkg0RsKw3dGyn7Zdw51HsbCthUcta6gYuRpuDP+/w7pMaVwSgF3\n Bw==","X-Received":["by 2002:a05:690e:1401:b0:654:1b90:1682 with SMTP id\n 956f58d0204a3-6541b902d9fmr15367727d50.35.1776850139762;\n Wed, 22 Apr 2026 02:28:59 -0700 (PDT)","by 2002:a05:690e:1401:b0:654:1b90:1682 with SMTP id\n 956f58d0204a3-6541b902d9fmr15367722d50.35.1776850139377; Wed, 22 Apr 2026\n 02:28:59 -0700 (PDT)"],"MIME-Version":"1.0","References":"<20260422073435.140670-1-tkaminsk@redhat.com>","In-Reply-To":"<20260422073435.140670-1-tkaminsk@redhat.com>","From":"Jonathan Wakely <jwakely@redhat.com>","Date":"Wed, 22 Apr 2026 10:28:43 +0100","X-Gm-Features":"AQROBzB97e3RJ7V0t8vCCyQIXmji51G8nSH-FHGSKEPfAtPGAc5KJGkg4vGL--Q","Message-ID":"\n <CACb0b4m4vFBaF6oBN2Tj3Cp0kRTfoTMKRyedMJqb0PUY9=f-ng@mail.gmail.com>","Subject":"Re: [PATCH] libstdc++: Accept data_handle_type by value in mdspan\n deduction guide.","To":"=?utf-8?q?Tomasz_Kami=C5=84ski?= <tkaminsk@redhat.com>","Cc":"libstdc++@gcc.gnu.org, gcc-patches@gcc.gnu.org","X-Mimecast-Spam-Score":"0","X-Mimecast-MFC-PROC-ID":"xp5CObftswJUS0VPzE4SFue2LhHn4ra7GiSBxvoNvLM_1776850140","X-Mimecast-Originator":"redhat.com","Content-Type":"text/plain; charset=\"UTF-8\"","Content-Transfer-Encoding":"quoted-printable","X-BeenThere":"gcc-patches@gcc.gnu.org","X-Mailman-Version":"2.1.30","Precedence":"list","List-Id":"Gcc-patches mailing list <gcc-patches.gcc.gnu.org>","List-Unsubscribe":"<https://gcc.gnu.org/mailman/options/gcc-patches>,\n <mailto:gcc-patches-request@gcc.gnu.org?subject=unsubscribe>","List-Archive":"<https://gcc.gnu.org/pipermail/gcc-patches/>","List-Post":"<mailto:gcc-patches@gcc.gnu.org>","List-Help":"<mailto:gcc-patches-request@gcc.gnu.org?subject=help>","List-Subscribe":"<https://gcc.gnu.org/mailman/listinfo/gcc-patches>,\n <mailto:gcc-patches-request@gcc.gnu.org?subject=subscribe>","Errors-To":"gcc-patches-bounces~incoming=patchwork.ozlabs.org@gcc.gnu.org"}}]