This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PR64164] drop copyrename, integrate into expand
- From: Alexandre Oliva <aoliva at redhat dot com>
- To: Andreas Schwab <schwab at linux-m68k dot org>
- Cc: Christophe Lyon <christophe dot lyon at linaro dot org>, GCC Patches <gcc-patches at gcc dot gnu dot org>, Patrick Marlier <patrick dot marlier at gmail dot com>, Jeff Law <law at redhat dot com>, James Greenhalgh <james dot greenhalgh at arm dot com>, "H.J. Lu" <hjl dot tools at gmail dot com>, Segher Boessenkool <segher at kernel dot crashing dot org>, Richard Biener <richard dot guenther at gmail dot com>, David Edelsohn <dje dot gcc at gmail dot com>, Eric Botcazou <ebotcazou at adacore dot com>
- Date: Fri, 21 Aug 2015 04:57:19 -0300
- Subject: Re: [PR64164] drop copyrename, integrate into expand
- Authentication-results: sourceware.org; auth=none
- References: <orio9cw10j dot fsf at livre dot home> <orwpxqvqnp dot fsf at livre dot home> <20150723203112 dot GB27818 at gate dot crashing dot org> <CAMe9rOpR+2gPxo0tKaRPtcML_Q4=r-_=9iqk+_JZFPkM=eN=BQ at mail dot gmail dot com> <CAMe9rOpbLEyDexVJqJAFJ3W6o4AktNog-jwk2CY4GZkrmT+nfA at mail dot gmail dot com> <or4mkmhgc9 dot fsf at livre dot home> <CAMe9rOp=S5fu1N=i7waswCYqJeLBCrySqYdFYkVa7LV04vpQSg at mail dot gmail dot com> <CAMe9rOrq+ZBAg1nZ1twEcPqwBj4j9+XA+SXQJVWWzjfdvidjtw at mail dot gmail dot com> <or1tfkdjhj dot fsf at livre dot home> <20150810082355 dot GA31149 at arm dot com> <55C8BFC3 dot 3030603 at redhat dot com> <CAKQMxzRzMrGtf921vqXCno5uoBN+uzsnJ5wX2Twmvhp1ziAEcA at mail dot gmail dot com> <or37zlpujd dot fsf at livre dot home> <CAKdteOafwG_fm=U_MexgQ8-ep-5vQtbgoeoCtJ55QKpRim3+RQ at mail dot gmail dot com> <or614e2kkq dot fsf at livre dot home> <orbne51el4 dot fsf at livre dot home> <orh9nvzsgz dot fsf at livre dot home> <mvmpp2jcsv2 dot fsf at hawking dot suse dot de> <mvmlhd7cs21 dot fsf at hawking dot suse dot de> <or7foryy1m dot fsf at livre dot home> <orvbcaygjy dot fsf at livre dot home>
On Aug 19, 2015, Alexandre Oliva <aoliva@redhat.com> wrote:
> I have verified in the expand dumps that both the gimple and the rtl
> representation in the relevant parts of the code are identical, except
> for the presence of debug stmts and insns.
While comparing the dumps, I noticed -fdump-unnumbered-links no longer
worked like it did back when I introduced it, with the very purpose of
making it easier to compare dumps with and without debug insns.
When the insn_uid was moved out of the u[] array, the indices that
print-rtl tested to tell whether to omit the ids of the prev and next
insns got off by one.
This patch updates the test to match the current indices.
Bootstrapping on ia64-linux-gnu. Ok to install?
fix -fdump-unnumbered-links
From: Alexandre Oliva <aoliva@redhat.com>
for gcc/ChangeLog
* print-rtl.c (print_rtx): Check the correct range for
flag_dump_unnumbered_links to behave as documented.
---
gcc/print-rtl.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/gcc/print-rtl.c b/gcc/print-rtl.c
index aacadbb..b541d83 100644
--- a/gcc/print-rtl.c
+++ b/gcc/print-rtl.c
@@ -550,7 +550,7 @@ print_rtx (const_rtx in_rtx)
}
if (flag_dump_unnumbered
- || (flag_dump_unnumbered_links && (i == 1 || i == 2)
+ || (flag_dump_unnumbered_links && i <= 1
&& (INSN_P (in_rtx) || NOTE_P (in_rtx)
|| LABEL_P (in_rtx) || BARRIER_P (in_rtx))))
fputs (" #", outfile);
--
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 Brasil GNU Toolchain Engineer
- References:
- Re: [PR64164] drop copyrename, integrate into expand
- Re: [PR64164] drop copyrename, integrate into expand
- Re: [PR64164] drop copyrename, integrate into expand
- Re: [PR64164] drop copyrename, integrate into expand
- Re: [PR64164] drop copyrename, integrate into expand
- Re: [PR64164] drop copyrename, integrate into expand
- Re: [PR64164] drop copyrename, integrate into expand
- Re: [PR64164] drop copyrename, integrate into expand
- Re: [PR64164] drop copyrename, integrate into expand
- Re: [PR64164] drop copyrename, integrate into expand
- Re: [PR64164] drop copyrename, integrate into expand
- Re: [PR64164] drop copyrename, integrate into expand
- Re: [PR64164] drop copyrename, integrate into expand