Bug 51511 - [4.6/4.7 regression] gcc.dg/pr45819.c fails on arm-linux-gnueabi due to PR50078+PR51466 fix
Summary: [4.6/4.7 regression] gcc.dg/pr45819.c fails on arm-linux-gnueabi due to PR500...
Status: RESOLVED FIXED
Alias: None
Product: gcc
Classification: Unclassified
Component: testsuite (show other bugs)
Version: 4.6.3
: P3 normal
Target Milestone: 4.6.3
Assignee: Jakub Jelinek
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-12-12 08:20 UTC by Mikael Pettersson
Modified: 2011-12-12 19:45 UTC (History)
1 user (show)

See Also:
Host:
Target: arm-linux-gnueabi
Build:
Known to work: 4.6.2
Known to fail: 4.6.3, 4.7.0
Last reconfirmed: 2011-12-12 00:00:00


Attachments
gcc47-pr51511.patch (264 bytes, patch)
2011-12-12 09:13 UTC, Jakub Jelinek
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Mikael Pettersson 2011-12-12 08:20:23 UTC
The PR50078+PR51466 fix, r182157 on 4.6 branch, caused a test suite regression on arm-linux-gnueabi:

+FAIL: gcc.dg/pr45819.c (test for excess errors)

because the compiler now emits the following warning:

pr45819.c: In function 'ehci_hub_control':
pr45819.c:16:10: warning: mis-aligned access used for structure member [-fstrict-volatile-bitfields]
pr45819.c:16:10: note: when a volatile object spans multiple type-sized locations, the compiler must choose between using a single mis-aligned access to preserve the volatility, or using multiple aligned accesses to avoid runtime faults; this code may fail at runtime if the hardware does not allow this access

I believe the warning is correct, so perhaps the warning should just be suppressed
in pr45819.c with -w.  gcc-4.7-20111210 has the same issue.
Comment 1 Jakub Jelinek 2011-12-12 09:13:58 UTC
Created attachment 26052 [details]
gcc47-pr51511.patch

Yeah, I agree.
Comment 2 Jakub Jelinek 2011-12-12 16:48:56 UTC
Author: jakub
Date: Mon Dec 12 16:48:46 2011
New Revision: 182247

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=182247
Log:
	PR testsuite/51511
	* gcc.dg/pr45819.c: Add -w to dg-options.

Modified:
    trunk/gcc/testsuite/ChangeLog
    trunk/gcc/testsuite/gcc.dg/pr45819.c
Comment 3 Jakub Jelinek 2011-12-12 19:44:25 UTC
Author: jakub
Date: Mon Dec 12 19:44:22 2011
New Revision: 182261

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=182261
Log:
	PR testsuite/51511
	* gcc.dg/pr45819.c: Add -w to dg-options.

Modified:
    branches/gcc-4_6-branch/gcc/testsuite/ChangeLog
    branches/gcc-4_6-branch/gcc/testsuite/gcc.dg/pr45819.c
Comment 4 Jakub Jelinek 2011-12-12 19:45:36 UTC
Fixed.