--- Comment #7 from vmakarov at gcc dot gnu dot org 2009-03-12 14:40
---
Subject: Bug 39432
Author: vmakarov
Date: Thu Mar 12 14:39:55 2009
New Revision: 144812
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=144812
Log:
2009-03-12 Vladimir Makarov vmaka...@redhat.com
--- Comment #8 from jakub at gcc dot gnu dot org 2009-03-12 15:49 ---
Fixed, thanks.
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #1 from vmakarov at redhat dot com 2009-03-11 16:57 ---
Jakub, how is about the following patch. Is it ok for you? I mean correct
user variable identification.
2009-03-11 Vladimir Makarov vmaka...@redhat.com
PR debug/39432
* ira-int.h (struct allocno):
--- Comment #2 from rguenth at gcc dot gnu dot org 2009-03-11 17:02 ---
You should use DECL_ARTIFICIAL I think.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=39432
--- Comment #3 from vmakarov at redhat dot com 2009-03-11 17:10 ---
Thanks, Richard.
So instead of DECL_NAME (decl) != NULL I should use ! DECL_ARTIFICIAL
(decl). Right?
Ok, I'll test the new patch then and send it for approval after testing.
--
--- Comment #4 from jakub at gcc dot gnu dot org 2009-03-11 17:11 ---
Also perhaps should test DECL_HARD_REGISTER, for DECL_HARD_REGISTER we
shouldn't limit them in any way to allow the user to shoot himself.
In any case, I'll test your patch momentarily.
--
--- Comment #5 from vmakarov at redhat dot com 2009-03-11 17:28 ---
As for DECL_HARD_REGISTER, such decl regs are never considered by IRA for
allocation. So I think there is no necessity to check them here.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=39432
--- Comment #6 from jakub at gcc dot gnu dot org 2009-03-11 19:38 ---
I can confirm that trunk with the #c1 patch modified as mentioned in #c3 cures
all the gdb.base/store.exp failures.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=39432