On May 3, 2011, Revital1 Eres <e...@il.ibm.com> wrote:
> Please let me know if you need any further info.
No, thanks, that was all I needed.
I think this will restore proper functioning to SMS in the presence of
debug insns. A while ago, we'd never generate deps of non-debug insns
on debug insns. I introduced them to enable sched to adjust (reset)
debug insns when non-debug insns were moved before them. I believe it
is safe to leave them out of the SCCs. Even though this will end up
causing some loss of debug info, that's probably unavoidable, and the
end result after this change is pobably the best we can hope for. Your
thoughts?
Is this ok to install if it regstraps successfully?
for gcc/ChangeLog
from Alexandre Oliva <aol...@redhat.com>
* ddg.c (build_intra_loop_deps): Discard deps of nondebug on debug.
Index: gcc/ddg.c
===================================================================
--- gcc/ddg.c.orig 2011-05-04 03:42:20.938013725 -0300
+++ gcc/ddg.c 2011-05-04 03:42:30.202253457 -0300
@@ -452,7 +452,12 @@ build_intra_loop_deps (ddg_ptr g)
FOR_EACH_DEP (dest_node->insn, SD_LIST_BACK, sd_it, dep)
{
- ddg_node_ptr src_node = get_node_of_insn (g, DEP_PRO (dep));
+ ddg_node_ptr src_node;
+
+ if (DEBUG_INSN_P (DEP_PRO (dep)) && !DEBUG_INSN_P (dest_node->insn))
+ continue;
+
+ src_node = get_node_of_insn (g, DEP_PRO (dep));
if (!src_node)
continue;
--
Alexandre Oliva, freedom fighter http://FSFLA.org/~lxoliva/
You must be the change you wish to see in the world. -- Gandhi
Be Free! -- http://FSFLA.org/ FSF Latin America board member
Free Software Evangelist Red Hat Brazil Compiler Engineer
