This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [PATCH] 69517 - [5/6 regression] SEGV on a VLA with excess initializer elements

From: Martin Sebor <msebor at gmail dot com>
To: Jakub Jelinek <jakub at redhat dot com>
Cc: Andreas Schwab <schwab at suse dot de>, Jason Merrill <jason at redhat dot com>, Gcc Patch List <gcc-patches at gcc dot gnu dot org>
Date: Fri, 15 Apr 2016 08:33:28 -0600
Subject: Re: [PATCH] 69517 - [5/6 regression] SEGV on a VLA with excess initializer elements
Authentication-results: sourceware.org; auth=none
References: <56F2F2D8 dot 10708 at gmail dot com> <56F40CA1 dot 3060005 at redhat dot com> <56FEFE08 dot 8010207 at gmail dot com> <5702FA46 dot 9020807 at redhat dot com> <5706F5D5 dot 9030204 at gmail dot com> <570ADE43 dot 9080107 at gmail dot com> <570D3BC1 dot 1040608 at redhat dot com> <570E91B1 dot 90602 at gmail dot com> <mvmoa9cpj7k dot fsf at hawking dot suse dot de> <570FB693 dot 5080806 at gmail dot com> <20160415123111 dot GR19207 at tucnak dot redhat dot com>

On 04/15/2016 06:31 AM, Jakub Jelinek wrote:

On Thu, Apr 14, 2016 at 09:26:11AM -0600, Martin Sebor wrote:

/daten/aranym/gcc/gcc-20160414/gcc/testsuite/g++.dg/cpp1y/vla11.C: In instantiation of 'struct TestType<32u>':
/daten/aranym/gcc/gcc-20160414/gcc/testsuite/g++.dg/cpp1y/vla11.C:201:1:   required from here
/daten/aranym/gcc/gcc-20160414/gcc/testsuite/g++.dg/cpp1y/vla11.C:89:27: error: requested alignment 32 is larger than 16 [-Wattributes]


Thank you for the heads up (and sorry about the breakage).  I've
committed r234976 to fix that.


Probably because of this change you haven't reverted (removed) the vla11.C
testcase even when the ChangeLog said you've done that.
And, the testcase obviously fails on the trunk, so I've committed following
as obvious:


Probably.  I was rushing to unblock others whose bootstrap was
failing and didn't check carefully enough.  Thanks for fixing that.

Martin

References:
- Re: [PATCH] 69517 - [5/6 regression] SEGV on a VLA with excess initializer elements
  - From: Martin Sebor
- Re: [PATCH] 69517 - [5/6 regression] SEGV on a VLA with excess initializer elements
  - From: Jason Merrill
- Re: [PATCH] 69517 - [5/6 regression] SEGV on a VLA with excess initializer elements
  - From: Martin Sebor
- Re: [PATCH] 69517 - [5/6 regression] SEGV on a VLA with excess initializer elements
  - From: Martin Sebor
- Re: [PATCH] 69517 - [5/6 regression] SEGV on a VLA with excess initializer elements
  - From: Jason Merrill
- Re: [PATCH] 69517 - [5/6 regression] SEGV on a VLA with excess initializer elements
  - From: Martin Sebor
- Re: [PATCH] 69517 - [5/6 regression] SEGV on a VLA with excess initializer elements
  - From: Andreas Schwab
- Re: [PATCH] 69517 - [5/6 regression] SEGV on a VLA with excess initializer elements
  - From: Martin Sebor
- Re: [PATCH] 69517 - [5/6 regression] SEGV on a VLA with excess initializer elements
  - From: Jakub Jelinek

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]