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: Richard Biener <richard dot guenther at gmail dot com>
- To: Alexandre Oliva <aoliva at redhat dot com>
- Cc: Andreas Schwab <schwab at linux-m68k dot org>, 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>, David Edelsohn <dje dot gcc at gmail dot com>, Eric Botcazou <ebotcazou at adacore dot com>
- Date: Fri, 21 Aug 2015 10:35:05 +0200
- 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> <oregixxee8 dot fsf at livre dot home>
On Fri, Aug 21, 2015 at 9:57 AM, Alexandre Oliva <aoliva@redhat.com> wrote:
> 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?
Ok.
Thanks,
Richard.
> 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
- Re: [PR64164] drop copyrename, integrate into expand