From 82bc5f13b8fc321306f1fff5ec3ca3fc1ef8c940 Mon Sep 17 00:00:00 2001 From: redi Date: Sat, 2 May 2015 18:19:39 +0000 Subject: [PATCH] * include/experimental/any (any::_Storage): Fix alignment of buffer. (any::_Internal): Check alignment of type. * testsuite/experimental/any/cons/aligned.cc: New. * testsuite/experimental/any/misc/any_cast_neg.cc: Adjust dg-error. git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@222729 138bc75d-0d04-0410-961f-82ee72b054a4 --- libstdc++-v3/ChangeLog | 5 +++ libstdc++-v3/include/experimental/any | 5 ++- .../testsuite/experimental/any/cons/aligned.cc | 52 ++++++++++++++++++++++ .../experimental/any/misc/any_cast_neg.cc | 2 +- 4 files changed, 61 insertions(+), 3 deletions(-) create mode 100644 libstdc++-v3/testsuite/experimental/any/cons/aligned.cc diff --git a/libstdc++-v3/ChangeLog b/libstdc++-v3/ChangeLog index 7c8ed664547..f04b97a4f5f 100644 --- a/libstdc++-v3/ChangeLog +++ b/libstdc++-v3/ChangeLog @@ -1,5 +1,10 @@ 2015-05-02 Jonathan Wakely + * include/experimental/any (any::_Storage): Fix alignment of buffer. + (any::_Internal): Check alignment of type. + * testsuite/experimental/any/cons/aligned.cc: New. + * testsuite/experimental/any/misc/any_cast_neg.cc: Adjust dg-error. + * include/experimental/iterator (ostream_joiner): Simplify by using the injected-class-name and the ostream_type typedef. diff --git a/libstdc++-v3/include/experimental/any b/libstdc++-v3/include/experimental/any index b2d1b9c1e48..7b5e5ecbcba 100644 --- a/libstdc++-v3/include/experimental/any +++ b/libstdc++-v3/include/experimental/any @@ -98,11 +98,12 @@ _GLIBCXX_BEGIN_NAMESPACE_VERSION _Storage& operator=(const _Storage&) = delete; void* _M_ptr; - std::aligned_storage::type _M_buffer; + aligned_storage::type _M_buffer; }; template, - bool _Fits = (sizeof(_Tp) <= sizeof(_Storage))> + bool _Fits = (sizeof(_Tp) <= sizeof(_Storage)) + && (alignof(_Tp) <= alignof(_Storage))> using _Internal = std::integral_constant; template diff --git a/libstdc++-v3/testsuite/experimental/any/cons/aligned.cc b/libstdc++-v3/testsuite/experimental/any/cons/aligned.cc new file mode 100644 index 00000000000..3923994038c --- /dev/null +++ b/libstdc++-v3/testsuite/experimental/any/cons/aligned.cc @@ -0,0 +1,52 @@ +// Copyright (C) 2015 Free Software Foundation, Inc. +// +// This file is part of the GNU ISO C++ Library. This library is free +// software; you can redistribute it and/or modify it under the +// terms of the GNU General Public License as published by the +// Free Software Foundation; either version 3, or (at your option) +// any later version. + +// This library is distributed in the hope that it will be useful, +// but WITHOUT ANY WARRANTY; without even the implied warranty of +// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +// GNU General Public License for more details. + +// You should have received a copy of the GNU General Public License along +// with this library; see the file COPYING3. If not see +// . + +// { dg-options "-std=gnu++14" } + +#include +#include +#include + +// Alignment requiremnts of this type prevent it being stored in 'any' +struct alignas(2 * alignof(void*)) X { }; + +bool +stored_internally(void* obj, const std::experimental::any& a) +{ + std::uintptr_t a_addr = reinterpret_cast(&a); + std::uintptr_t a_end = a_addr + sizeof(a); + std::uintptr_t obj_addr = reinterpret_cast(obj); + return (a_addr <= obj_addr) && (obj_addr < a_end); +} + +void +test01() +{ + std::experimental::any a = X{}; + X& x = std::experimental::any_cast(a); + VERIFY( !stored_internally(&x, a) ); + + a = 'X'; + char& c = std::experimental::any_cast(a); + VERIFY( stored_internally(&c, a) ); +} + +int +main() +{ + test01(); +} diff --git a/libstdc++-v3/testsuite/experimental/any/misc/any_cast_neg.cc b/libstdc++-v3/testsuite/experimental/any/misc/any_cast_neg.cc index 5823175cbeb..39e322642aa 100644 --- a/libstdc++-v3/testsuite/experimental/any/misc/any_cast_neg.cc +++ b/libstdc++-v3/testsuite/experimental/any/misc/any_cast_neg.cc @@ -26,5 +26,5 @@ void test01() using std::experimental::any_cast; const any y(1); - any_cast(y); // { dg-error "qualifiers" "" { target { *-*-* } } 355 } + any_cast(y); // { dg-error "qualifiers" "" { target { *-*-* } } 356 } } -- 2.11.4.GIT