http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54971
--- Comment #17 from Martin Jambor jamborm at gcc dot gnu.org 2013-01-04
14:41:08 UTC ---
(In reply to comment #13)
So, beyond the creation of new debug only accesses for whole struct writes
into
hole if there aren't too many holes, I
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54971
--- Comment #16 from Jakub Jelinek jakub at gcc dot gnu.org 2012-11-05
14:36:52 UTC ---
Author: jakub
Date: Mon Nov 5 14:36:47 2012
New Revision: 193162
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=193162
Log:
PR
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54971
--- Comment #13 from Jakub Jelinek jakub at gcc dot gnu.org 2012-10-29
08:36:03 UTC ---
So, beyond the creation of new debug only accesses for whole struct writes into
hole if there aren't too many holes, I wonder if SRA doesn't have
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54971
--- Comment #14 from Richard Biener rguenth at gcc dot gnu.org 2012-10-29
15:19:53 UTC ---
I think it has something like that, but it only is effective if there is any
scalarization and the intermediate copies are turned into dead code this
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54971
--- Comment #15 from Jakub Jelinek jakub at gcc dot gnu.org 2012-10-29
15:25:36 UTC ---
The intermediate copies should be DSE-able, as shown when the #c14 testcases
are changed to have a resp. x on all RHSs.
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54971
Jakub Jelinek jakub at gcc dot gnu.org changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54971
Martin Jambor jamborm at gcc dot gnu.org changed:
What|Removed |Added
URL|
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54971
--- Comment #10 from Martin Jambor jamborm at gcc dot gnu.org 2012-10-26
16:13:08 UTC ---
Author: jamborm
Date: Fri Oct 26 16:13:00 2012
New Revision: 192848
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=192848
Log:
2012-10-26
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54971
--- Comment #11 from Jakub Jelinek jakub at gcc dot gnu.org 2012-10-26
19:19:30 UTC ---
Author: jakub
Date: Fri Oct 26 19:19:25 2012
New Revision: 192860
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=192860
Log:
PR
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54971
--- Comment #4 from Martin Jambor jamborm at gcc dot gnu.org 2012-10-22
14:55:35 UTC ---
Unfortunately, the patch causes -fcompare-debug issues. The problem
is that with it we create some declarations only when producing debug
info which
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54971
--- Comment #5 from Jakub Jelinek jakub at gcc dot gnu.org 2012-10-22
15:09:05 UTC ---
Can you say what -fcompare-debug failures you saw (or was it a bootstrap
problem already)?
Generally, differences in DECL_UIDs between -g and -g0 should
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54971
--- Comment #6 from Martin Jambor jamborm at gcc dot gnu.org 2012-10-22
15:14:13 UTC ---
(In reply to comment #5)
Can you say what -fcompare-debug failures you saw (or was it a bootstrap
problem already)?
Bootstrap actually passes.
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54971
--- Comment #7 from Jakub Jelinek jakub at gcc dot gnu.org 2012-10-22
16:19:18 UTC ---
Created attachment 28510
-- http://gcc.gnu.org/bugzilla/attachment.cgi?id=28510
gcc48-pr54971-incremental.patch
Incremental patch that makes the
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54971
--- Comment #8 from Jakub Jelinek jakub at gcc dot gnu.org 2012-10-22
19:59:44 UTC ---
With your patch and my incremental patch on top of it bootstrap/regtest
passed on both x86_64-linux and i686-linux btw.
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54971
--- Comment #3 from Martin Jambor jamborm at gcc dot gnu.org 2012-10-19
16:01:20 UTC ---
Created attachment 28493
-- http://gcc.gnu.org/bugzilla/attachment.cgi?id=28493
Untested patch
I'm currently bootstrapping and testing this
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54971
--- Comment #1 from Jakub Jelinek jakub at gcc dot gnu.org 2012-10-18
13:35:05 UTC ---
From quick skimming of tree-sra.c, I'd say we could add another bool flag like
grp_to_be_replaced (say grp_to_be_debug_replaced), and in the else block
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54971
--- Comment #2 from Martin Jambor jamborm at gcc dot gnu.org 2012-10-18
17:37:25 UTC ---
I already have a work-in-progress patch based on your suggestions that
works for the testcase but need to think a bit more about less obvious
cases
17 matches
Mail list logo
