public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
From: Eric Botcazou <ebotcazou@adacore.com>
To: Alexandre Oliva <aoliva@redhat.com>
Cc: gcc-patches@gcc.gnu.org
Subject: Re: fix left-over debug insns in DCE
Date: Fri, 03 Jun 2011 11:06:00 -0000	[thread overview]
Message-ID: <201106031258.13095.ebotcazou@adacore.com> (raw)
In-Reply-To: <orei3boh19.fsf@livre.localdomain>

> Hmm...  Maybe it could, I'm not sure.  The problem is that DCE removes
> insns, and then DF associates remaining uses in debug insns to earlier
> DEFs.  Adjusting debug insns in DCE is right per the VTA design motto:
> decide as if debug insns weren't there, adjust them as you would adjust
> non-debug insns.  This code borrowed from DF into DCE is the “adjust”
> bit.

But DCE isn't the only pass that removes insns.  Does the same logic need to be 
replicated in all passes that do?  On the other hand, these passes call into 
DF when they remove insns, so DF is a central place here.

> Err...  These depend on the interface changes of functions defined
> within DF to work.

No, they don't, I can compile them independently.

-- 
Eric Botcazou

  reply	other threads:[~2011-06-03 11:06 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-05-30 12:15 Alexandre Oliva
2011-06-02 15:26 ` Eric Botcazou
2011-06-03  1:16   ` Alexandre Oliva
2011-06-03 11:06     ` Eric Botcazou [this message]
2011-06-06  5:38       ` Alexandre Oliva
2011-06-06  6:05         ` Jakub Jelinek
2011-06-06  8:00         ` Eric Botcazou
2011-06-06 10:07           ` Alexandre Oliva
2011-06-06 13:28           ` Alexandre Oliva
2011-06-07 12:25             ` [PATCH] Fix ICE in reset_unmarked_insns_debug_uses (PR middle-end/49308) Jakub Jelinek
2011-06-09  8:04               ` Jakub Jelinek
2012-04-09  6:14             ` fix left-over debug insns in DCE Alexandre Oliva
2012-04-09 13:42               ` Eric Botcazou
2012-04-13 15:59                 ` Alexandre Oliva
2012-04-13 16:45                   ` Paolo Bonzini
2012-05-03 18:27                     ` [PR52983] eliminate autoinc from debug_insn locs (was: Re: fix left-over debug insns in DCE) Alexandre Oliva
2012-05-03 22:40                       ` [PR52983] eliminate autoinc from debug_insn locs Alexandre Oliva
2012-06-13  8:01                         ` Alexandre Oliva
2012-07-02 12:29                           ` Alexandre Oliva
2012-05-03 22:46                       ` [PR52983, PR48866] " Alexandre Oliva
2012-06-13  8:56                         ` Alexandre Oliva
2012-06-13  9:15                           ` Jakub Jelinek
2011-06-06  7:11       ` fix left-over debug insns in DCE Alexandre Oliva

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=201106031258.13095.ebotcazou@adacore.com \
    --to=ebotcazou@adacore.com \
    --cc=aoliva@redhat.com \
    --cc=gcc-patches@gcc.gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).