Re: [PATCH] make RTL/TREE/IPA dump kind an index

2017-05-17 Thread Richard Biener
On Sun, May 14, 2017 at 3:57 AM, Nathan Sidwell wrote: > On 05/13/2017 03:58 AM, Bernhard Reutner-Fischer wrote: > >> Specifically >> https://gcc.gnu.org/ml/fortran/2012-03/msg00094.html >> >> dejagnu-1.5.2 contains the libdirs tweak and was released 2015-01-30 >> >> Mike, can we please bump the

Re: [PATCH] make RTL/TREE/IPA dump kind an index

2017-05-13 Thread Nathan Sidwell
On 05/13/2017 03:58 AM, Bernhard Reutner-Fischer wrote: Specifically https://gcc.gnu.org/ml/fortran/2012-03/msg00094.html dejagnu-1.5.2 contains the libdirs tweak and was released 2015-01-30 Mike, can we please bump the required dejagnu version for GCC-8? Thanks There's also the version of

Re: [PATCH] make RTL/TREE/IPA dump kind an index

2017-05-13 Thread Bernhard Reutner-Fischer
On 12 May 2017 01:10:19 CEST, Bernhard Reutner-Fischer wrote: >On 12 May 2017 00:57:58 CEST, Rainer Orth >wrote: >>Bernhard Reutner-Fischer writes: >> >>> On 11 May 2017 15:10:36 CEST, Nathan Sidwell wrote: On 05/11/2017 08:28 AM, Rainer Orth wrote: > On the gcc-7 branch, the dump

Re: [PATCH] make RTL/TREE/IPA dump kind an index

2017-05-12 Thread Rainer Orth
Hi Nathan, > On 05/11/2017 06:57 PM, Rainer Orth wrote: > er, the introduction of scanlang.exp broke libatomic, libgomp, and >> libitm testing. Their logfiles show >> >> trunk/12-gcc/build/sparc-sun-solaris2.12/libatomic/testsuite/libatomic.log:ERROR: >> Couldn't find library file scanlang.exp. >

Re: [PATCH] make RTL/TREE/IPA dump kind an index

2017-05-11 Thread Bernhard Reutner-Fischer
On 12 May 2017 02:29:58 CEST, Nathan Sidwell wrote: >On 05/11/2017 06:52 PM, Bernhard Reutner-Fischer wrote: >> On 11 May 2017 15:10:36 CEST, Nathan Sidwell wrote: > >> Anyway, please also adjust the dump cleanup to handle class or I can >take care of it sometimes later. > >Applied this, which se

Re: [PATCH] make RTL/TREE/IPA dump kind an index

2017-05-11 Thread Nathan Sidwell
On 05/11/2017 06:52 PM, Bernhard Reutner-Fischer wrote: On 11 May 2017 15:10:36 CEST, Nathan Sidwell wrote: Anyway, please also adjust the dump cleanup to handle class or I can take care of it sometimes later. Applied this, which seems to do the trick. nathan -- Nathan Sidwell 2017-05-1

Re: [PATCH] make RTL/TREE/IPA dump kind an index

2017-05-11 Thread Nathan Sidwell
On 05/11/2017 06:57 PM, Rainer Orth wrote: er, the introduction of scanlang.exp broke libatomic, libgomp, and libitm testing. Their logfiles show trunk/12-gcc/build/sparc-sun-solaris2.12/libatomic/testsuite/libatomic.log:ERROR: Couldn't find library file scanlang.exp. trunk/12-gcc/build/sparc-

Re: [PATCH] make RTL/TREE/IPA dump kind an index

2017-05-11 Thread Bernhard Reutner-Fischer
On 12 May 2017 00:57:58 CEST, Rainer Orth wrote: >Bernhard Reutner-Fischer writes: > >> On 11 May 2017 15:10:36 CEST, Nathan Sidwell wrote: >>>On 05/11/2017 08:28 AM, Rainer Orth wrote: >>> On the gcc-7 branch, the dump file (from -fdump-class-hierarchy) >was called covariant7.C.002t.c

Re: [PATCH] make RTL/TREE/IPA dump kind an index

2017-05-11 Thread Rainer Orth
Bernhard Reutner-Fischer writes: > On 11 May 2017 15:10:36 CEST, Nathan Sidwell wrote: >>On 05/11/2017 08:28 AM, Rainer Orth wrote: >> >>> On the gcc-7 branch, the dump file (from -fdump-class-hierarchy) was >>> called covariant7.C.002t.class, now it's covariant7.C.002l.class. >>But >>> gcc/tes

Re: [PATCH] make RTL/TREE/IPA dump kind an index

2017-05-11 Thread Bernhard Reutner-Fischer
On 11 May 2017 15:10:36 CEST, Nathan Sidwell wrote: >On 05/11/2017 08:28 AM, Rainer Orth wrote: > >> On the gcc-7 branch, the dump file (from -fdump-class-hierarchy) was >> called covariant7.C.002t.class, now it's covariant7.C.002l.class. >But >> gcc/testsuite/lib/scantree.exp (scan-tree-dump) ex

Re: [PATCH] make RTL/TREE/IPA dump kind an index

2017-05-11 Thread Uros Bizjak
Hello! > Currently, the TDF_foo flags serve 3 purposes: > 1) what kind of dump > 2) how detailed to print it > 3) auxiliary message control "-da" option now dumps all IPA dumps in addition to RTL dumps. Can this behavior be reverted, so "-da" will dump only RTL dumps, as it was before your patch?

Re: [PATCH] make RTL/TREE/IPA dump kind an index

2017-05-11 Thread Nathan Sidwell
On 05/11/2017 08:28 AM, Rainer Orth wrote: On the gcc-7 branch, the dump file (from -fdump-class-hierarchy) was called covariant7.C.002t.class, now it's covariant7.C.002l.class. But gcc/testsuite/lib/scantree.exp (scan-tree-dump) expects the NNNt. form. fixed thusly, applied as obvious. the

Re: [PATCH] make RTL/TREE/IPA dump kind an index

2017-05-11 Thread Nathan Sidwell
On 05/11/2017 08:28 AM, Rainer Orth wrote: Hi Nathan, UNRESOLVED: g++.dg/inherit/covariant7.C -std=gnu++98 scan-tree-dump class "24 .*c6::_ZTcv0_n16_v0_n12_NV2c62f2Ev" UNRESOLVED: g++.dg/inherit/covariant7.C -std=gnu++98 scan-tree-dump class "28 .*c6::f2" UNRESOLVED: g++.dg/inherit

Re: [PATCH] make RTL/TREE/IPA dump kind an index

2017-05-11 Thread Rainer Orth
Hi Nathan, > On 05/10/2017 05:05 AM, Richard Biener wrote: >> On Tue, May 9, 2017 at 9:00 PM, Nathan Sidwell wrote: > >> + >> +#define TDF_ADDRESS(1 << 3)/* dump node addresses */ >> >> this leaves 1 << 2 unused. > > Yes, that was intentional (though I suspect my note about it was hid

Re: [PATCH] make RTL/TREE/IPA dump kind an index

2017-05-10 Thread Nathan Sidwell
On 05/10/2017 05:05 AM, Richard Biener wrote: On Tue, May 9, 2017 at 9:00 PM, Nathan Sidwell wrote: + +#define TDF_ADDRESS(1 << 3)/* dump node addresses */ this leaves 1 << 2 unused. Yes, that was intentional (though I suspect my note about it was hidden). As you say, I expec

Re: [PATCH] make RTL/TREE/IPA dump kind an index

2017-05-10 Thread Richard Biener
On Tue, May 9, 2017 at 9:00 PM, Nathan Sidwell wrote: > Currently, the TDF_foo flags serve 3 purposes: > 1) what kind of dump > 2) how detailed to print it > 3) auxiliary message control > > This addresses #1, which currently uses a bit mask of TDF_{TREE,RTL,IPA}, of > which exactly one must be se

[PATCH] make RTL/TREE/IPA dump kind an index

2017-05-09 Thread Nathan Sidwell
Currently, the TDF_foo flags serve 3 purposes: 1) what kind of dump 2) how detailed to print it 3) auxiliary message control This addresses #1, which currently uses a bit mask of TDF_{TREE,RTL,IPA}, of which exactly one must be set. The patch changes things so that these are now an index value