[Bug debug/43177] Handle at least simple cases of reversible insns in debug info

2010-03-02 Thread jakub at gcc dot gnu dot org
--- Comment #7 from jakub at gcc dot gnu dot org 2010-03-03 07:03 --- Fixed. -- jakub at gcc dot gnu dot org changed: What|Removed |Added Status|ASSIGNED

[Bug debug/43177] Handle at least simple cases of reversible insns in debug info

2010-03-02 Thread jakub at gcc dot gnu dot org
--- Comment #6 from jakub at gcc dot gnu dot org 2010-03-03 07:01 --- Subject: Bug 43177 Author: jakub Date: Wed Mar 3 07:01:27 2010 New Revision: 157189 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=157189 Log: PR debug/43177 * var-tracking.c (loc_cmp): Don't

[Bug debug/43177] Handle at least simple cases of reversible insns in debug info

2010-03-02 Thread jakub at gcc dot gnu dot org
--- Comment #5 from jakub at gcc dot gnu dot org 2010-03-03 06:59 --- Subject: Bug 43177 Author: jakub Date: Wed Mar 3 06:59:09 2010 New Revision: 157188 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=157188 Log: PR debug/43177 * var-tracking.c (loc_cmp): Don't

[Bug debug/43177] Handle at least simple cases of reversible insns in debug info

2010-03-02 Thread jakub at gcc dot gnu dot org
--- Comment #4 from jakub at gcc dot gnu dot org 2010-03-02 12:34 --- Updated patch which actually bootstraps/regtests posted: http://gcc.gnu.org/ml/gcc-patches/2010-03/msg00096.html -- jakub at gcc dot gnu dot org changed: What|Removed |Added

[Bug debug/43177] Handle at least simple cases of reversible insns in debug info

2010-03-02 Thread jakub at gcc dot gnu dot org
--- Comment #3 from jakub at gcc dot gnu dot org 2010-03-02 08:04 --- Unfortunately, the patch didn't bootstrap. One issue is loc_cmp, which asserts everywhere the modes are the same. Which isn't necessarily true, we can end up comparing e.g.: (subreg:QI (value/s/u:DI 28:4092 @0x4828ae8

[Bug debug/43177] Handle at least simple cases of reversible insns in debug info

2010-03-01 Thread jakub at gcc dot gnu dot org
--- Comment #2 from jakub at gcc dot gnu dot org 2010-03-01 16:25 --- Created an attachment (id=2) --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=2&action=view) gcc45-pr43177.patch Untested patch I'm playing with. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43177

[Bug debug/43177] Handle at least simple cases of reversible insns in debug info

2010-02-26 Thread jakub at gcc dot gnu dot org
--- Comment #1 from jakub at gcc dot gnu dot org 2010-02-26 14:07 --- The initial question is whether to implement these for easily reversible insns in vt_initialize, or when a VALUE becomes dead because nothing uses it. Implementing it in vt_initialize (add_stores and count_uses) would