* [v3] xfail experimental/memory_resource/resource_adaptor.cc on 32-bit Solaris/x86 (PR libstdc++/77691)
@ 2018-03-22 10:04 Rainer Orth
2018-03-22 11:56 ` Jonathan Wakely
0 siblings, 1 reply; 4+ messages in thread
From: Rainer Orth @ 2018-03-22 10:04 UTC (permalink / raw)
To: gcc-patches; +Cc: libstdc++
[-- Attachment #1: Type: text/plain, Size: 1037 bytes --]
experimental/memory_resource/resource_adaptor.cc currently FAILs on
32-bit Solaris/x86:
Assertion failed: aligned<max_align_t>(p), file /vol/gcc/src/hg/trunk/local/libstdc++-v3/testsuite/experimental/memory_resource/resource_adaptor.cc, line 56, function test05
As detailed in the PR, this happens because Solaris <stddef.h> and the
Studio 12.6 compilers know nothing about __float128/_Float128 and malloc
only returns pointers aligned to 8 bytes, not 16 bytes as expected by
gcc/ginclude/stddef.h (max_align_t).
Thus, while checking if Solaris i386 malloc can be changed here, I'm
xfailing the test.
Tested on i386-pc-solaris2.11, amd64-pc-solaris2.11, and
sparc-sun-solaris2.11.
Ok for mainline?
Rainer
--
-----------------------------------------------------------------------------
Rainer Orth, Center for Biotechnology, Bielefeld University
2018-03-22 Rainer Orth <ro@CeBiTec.Uni-Bielefeld.DE>
PR libstdc++/77691
* testsuite/experimental/memory_resource/resource_adaptor.cc:
xfail execution on 32-bit Solaris/x86.
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: solx86-v3-max_align_t-malloc.patch --]
[-- Type: text/x-patch, Size: 702 bytes --]
# HG changeset patch
# Parent 0d630469e1009ecfe592a698e6e006be425518b8
xfail experimental/memory_resource/resource_adaptor.cc on 32-bit Solaris/x86 (PR libstdc++/77691)
diff --git a/libstdc++-v3/testsuite/experimental/memory_resource/resource_adaptor.cc b/libstdc++-v3/testsuite/experimental/memory_resource/resource_adaptor.cc
--- a/libstdc++-v3/testsuite/experimental/memory_resource/resource_adaptor.cc
+++ b/libstdc++-v3/testsuite/experimental/memory_resource/resource_adaptor.cc
@@ -1,4 +1,5 @@
// { dg-do run { target c++14 } }
+// { dg-xfail-run-if "PR libstdc++/77691" { { i?86-*-solaris2.* x86_64-*-solaris2.* } && ilp32 } }
// Copyright (C) 2016-2018 Free Software Foundation, Inc.
//
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [v3] xfail experimental/memory_resource/resource_adaptor.cc on 32-bit Solaris/x86 (PR libstdc++/77691)
2018-03-22 10:04 [v3] xfail experimental/memory_resource/resource_adaptor.cc on 32-bit Solaris/x86 (PR libstdc++/77691) Rainer Orth
@ 2018-03-22 11:56 ` Jonathan Wakely
2018-03-22 13:42 ` Rainer Orth
0 siblings, 1 reply; 4+ messages in thread
From: Jonathan Wakely @ 2018-03-22 11:56 UTC (permalink / raw)
To: Rainer Orth; +Cc: gcc-patches, libstdc++
On 22/03/18 11:03 +0100, Rainer Orth wrote:
>experimental/memory_resource/resource_adaptor.cc currently FAILs on
>32-bit Solaris/x86:
>
>Assertion failed: aligned<max_align_t>(p), file /vol/gcc/src/hg/trunk/local/libstdc++-v3/testsuite/experimental/memory_resource/resource_adaptor.cc, line 56, function test05
>
>As detailed in the PR, this happens because Solaris <stddef.h> and the
>Studio 12.6 compilers know nothing about __float128/_Float128 and malloc
>only returns pointers aligned to 8 bytes, not 16 bytes as expected by
>gcc/ginclude/stddef.h (max_align_t).
>
>Thus, while checking if Solaris i386 malloc can be changed here, I'm
>xfailing the test.
>
>Tested on i386-pc-solaris2.11, amd64-pc-solaris2.11, and
>sparc-sun-solaris2.11.
>
>Ok for mainline?
OK, thanks.
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [v3] xfail experimental/memory_resource/resource_adaptor.cc on 32-bit Solaris/x86 (PR libstdc++/77691)
2018-03-22 11:56 ` Jonathan Wakely
@ 2018-03-22 13:42 ` Rainer Orth
2018-03-22 14:23 ` Jonathan Wakely
0 siblings, 1 reply; 4+ messages in thread
From: Rainer Orth @ 2018-03-22 13:42 UTC (permalink / raw)
To: Jonathan Wakely; +Cc: gcc-patches, libstdc++
Hi Jonathan,
> On 22/03/18 11:03 +0100, Rainer Orth wrote:
>>experimental/memory_resource/resource_adaptor.cc currently FAILs on
>>32-bit Solaris/x86:
>>
>>Assertion failed: aligned<max_align_t>(p), file
>> /vol/gcc/src/hg/trunk/local/libstdc++-v3/testsuite/experimental/memory_resource/resource_adaptor.cc,
>> line 56, function test05
>>
>>As detailed in the PR, this happens because Solaris <stddef.h> and the
>>Studio 12.6 compilers know nothing about __float128/_Float128 and malloc
>>only returns pointers aligned to 8 bytes, not 16 bytes as expected by
>>gcc/ginclude/stddef.h (max_align_t).
>>
>>Thus, while checking if Solaris i386 malloc can be changed here, I'm
>>xfailing the test.
>>
>>Tested on i386-pc-solaris2.11, amd64-pc-solaris2.11, and
>>sparc-sun-solaris2.11.
>>
>>Ok for mainline?
>
> OK, thanks.
installed on mainline. I just noticed that the bug is marked as 7
regression; I guess it's ok to backport to the gcc-7 branch, too?
Rainer
--
-----------------------------------------------------------------------------
Rainer Orth, Center for Biotechnology, Bielefeld University
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [v3] xfail experimental/memory_resource/resource_adaptor.cc on 32-bit Solaris/x86 (PR libstdc++/77691)
2018-03-22 13:42 ` Rainer Orth
@ 2018-03-22 14:23 ` Jonathan Wakely
0 siblings, 0 replies; 4+ messages in thread
From: Jonathan Wakely @ 2018-03-22 14:23 UTC (permalink / raw)
To: Rainer Orth; +Cc: Jonathan Wakely, gcc-patches, libstdc++
On 22 March 2018 at 13:42, Rainer Orth wrote:
> Hi Jonathan,
>
>> On 22/03/18 11:03 +0100, Rainer Orth wrote:
>>>experimental/memory_resource/resource_adaptor.cc currently FAILs on
>>>32-bit Solaris/x86:
>>>
>>>Assertion failed: aligned<max_align_t>(p), file
>>> /vol/gcc/src/hg/trunk/local/libstdc++-v3/testsuite/experimental/memory_resource/resource_adaptor.cc,
>>> line 56, function test05
>>>
>>>As detailed in the PR, this happens because Solaris <stddef.h> and the
>>>Studio 12.6 compilers know nothing about __float128/_Float128 and malloc
>>>only returns pointers aligned to 8 bytes, not 16 bytes as expected by
>>>gcc/ginclude/stddef.h (max_align_t).
>>>
>>>Thus, while checking if Solaris i386 malloc can be changed here, I'm
>>>xfailing the test.
>>>
>>>Tested on i386-pc-solaris2.11, amd64-pc-solaris2.11, and
>>>sparc-sun-solaris2.11.
>>>
>>>Ok for mainline?
>>
>> OK, thanks.
>
> installed on mainline. I just noticed that the bug is marked as 7
> regression; I guess it's ok to backport to the gcc-7 branch, too?
Yes please.
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2018-03-22 14:23 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-03-22 10:04 [v3] xfail experimental/memory_resource/resource_adaptor.cc on 32-bit Solaris/x86 (PR libstdc++/77691) Rainer Orth
2018-03-22 11:56 ` Jonathan Wakely
2018-03-22 13:42 ` Rainer Orth
2018-03-22 14:23 ` Jonathan Wakely
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).