[Bug tree-optimization/53438] [4.7/4.8 Regression] Bitfield store replaced with full-byte store

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53438 --- Comment #15 from William J. Schmidt wschmidt at gcc dot gnu.org 2012-05-28 13:58:30 UTC --- Author: wschmidt Date: Mon May 28 13:58:18 2012 New Revision: 187930 URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=187930 Log: 2012-05-28 Bill

[Bug tree-optimization/53438] [4.7/4.8 Regression] Bitfield store replaced with full-byte store

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53438 William J. Schmidt wschmidt at gcc dot gnu.org changed: What|Removed |Added Status|ASSIGNED|RESOLVED

[Bug tree-optimization/53438] [4.7/4.8 Regression] Bitfield store replaced with full-byte store

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53438 Jakub Jelinek jakub at gcc dot gnu.org changed: What|Removed |Added CC||rockeet at gmail

[Bug tree-optimization/53438] [4.7/4.8 Regression] Bitfield store replaced with full-byte store

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53438 --- Comment #18 from Jakub Jelinek jakub at gcc dot gnu.org 2012-05-28 14:34:30 UTC --- (In reply to comment #14) Anyway, would it be possible to have a testcase for the gcc testsuite too? I've been trying, but so far haven't been able to

[Bug tree-optimization/53438] [4.7/4.8 Regression] Bitfield store replaced with full-byte store

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53438 --- Comment #19 from William J. Schmidt wschmidt at gcc dot gnu.org 2012-05-28 22:31:40 UTC --- Excellent. Thanks, Jakub.

[Bug tree-optimization/53438] [4.7/4.8 Regression] Bitfield store replaced with full-byte store

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53438 Jakub Jelinek jakub at gcc dot gnu.org changed: What|Removed |Added CC||jakub at gcc dot

[Bug tree-optimization/53438] [4.7/4.8 Regression] Bitfield store replaced with full-byte store

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53438 --- Comment #14 from William J. Schmidt wschmidt at gcc dot gnu.org 2012-05-25 15:37:27 UTC --- (In reply to comment #13) It didn't work because you've used the patch as svn commit message instead of ChangeLog entry. Whoops, found a new kind

[Bug tree-optimization/53438] [4.7/4.8 Regression] Bitfield store replaced with full-byte store

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53438 --- Comment #9 from rguenther at suse dot de rguenther at suse dot de 2012-05-23 07:04:51 UTC --- On Tue, 22 May 2012, wschmidt at gcc dot gnu.org wrote: http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53438 --- Comment #8 from William J.

[Bug tree-optimization/53438] [4.7/4.8 Regression] Bitfield store replaced with full-byte store

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53438 --- Comment #10 from William J. Schmidt wschmidt at gcc dot gnu.org 2012-05-23 12:35:11 UTC --- Sure. My testing was on 4.7, so I'll sanity-test trunk first. I can commit the backport next week also, assuming no fallout between now and then.

[Bug tree-optimization/53438] [4.7/4.8 Regression] Bitfield store replaced with full-byte store

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53438 --- Comment #11 from rguenther at suse dot de rguenther at suse dot de 2012-05-23 12:39:31 UTC --- On Wed, 23 May 2012, wschmidt at gcc dot gnu.org wrote: http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53438 --- Comment #10 from William J.

[Bug tree-optimization/53438] [4.7/4.8 Regression] Bitfield store replaced with full-byte store

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53438 --- Comment #12 from William J. Schmidt wschmidt at gcc dot gnu.org 2012-05-23 17:38:44 UTC --- Hm, the auto-commenter isn't working. Committed to trunk: 2012-05-23 Richard Guenther rguent...@suse.de PR tree-optimization/53438 *

[Bug tree-optimization/53438] [4.7/4.8 Regression] Bitfield store replaced with full-byte store

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53438 Andrew Pinski pinskia at gcc dot gnu.org changed: What|Removed |Added Target Milestone|4.7.2 |4.7.1

[Bug tree-optimization/53438] [4.7/4.8 Regression] Bitfield store replaced with full-byte store

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53438 Richard Guenther rguenth at gcc dot gnu.org changed: What|Removed |Added CC||jamborm at

[Bug tree-optimization/53438] [4.7/4.8 Regression] Bitfield store replaced with full-byte store

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53438 Richard Guenther rguenth at gcc dot gnu.org changed: What|Removed |Added Status|UNCONFIRMED |NEW Last

[Bug tree-optimization/53438] [4.7/4.8 Regression] Bitfield store replaced with full-byte store

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53438 Richard Guenther rguenth at gcc dot gnu.org changed: What|Removed |Added Status|NEW |ASSIGNED

[Bug tree-optimization/53438] [4.7/4.8 Regression] Bitfield store replaced with full-byte store

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53438 --- Comment #7 from William J. Schmidt wschmidt at gcc dot gnu.org 2012-05-22 13:13:04 UTC --- Yep, I'll check it out. Thanks, Richard!

[Bug tree-optimization/53438] [4.7/4.8 Regression] Bitfield store replaced with full-byte store

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53438 --- Comment #8 from William J. Schmidt wschmidt at gcc dot gnu.org 2012-05-22 18:04:21 UTC --- Richard, the patch looks good. It fixed the reduced test case, solves the customer problem, and bootstraps on powerpc64-linux with no regressions.

[Bug tree-optimization/53438] [4.7/4.8 Regression] Bitfield store replaced with full-byte store

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53438 Andrew Pinski pinskia at gcc dot gnu.org changed: What|Removed |Added Target Milestone|--- |4.7.2

[Bug tree-optimization/53438] [4.7/4.8 Regression] Bitfield store replaced with full-byte store

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53438 --- Comment #3 from Andrew Pinski pinskia at gcc dot gnu.org 2012-05-22 05:34:11 UTC --- I see what happens. I think SRA is taking the right hand side's type rather than look at the left hand side where it is a bitfield.