* Add support for dumping multiple dump files under one name
@ 2018-06-22 11:16 Jan Hubicka
2018-06-29 8:47 ` Richard Biener
0 siblings, 1 reply; 6+ messages in thread
From: Jan Hubicka @ 2018-06-22 11:16 UTC (permalink / raw)
To: gcc-patches, rguenther, dmalcolm
Hi,
this patch adds dumpfile support for dumps that come in multiple parts. This
is needed for WPA stream-out dump since we stream partitions in parallel and
the dumps would come up in random order. Parts are added by new parameter that
is initialzed by default to -1 (no parts).
One thing I skipped is any support for duplicated opening of file
with parts since I do not need it.
Bootstrapped/regtested x86_64-linux, OK?
Honza
* dumpfile.c (gcc::dump_manager::get_dump_file_name): Add PART
parameter.
(gcc::dump_manager::get_dump_file_name): likewise.
(dump_begin): Likewise.
* dumpfile.h (dump_begin): Update prototype.
(gcc::dump_manager::get_dump_file_name,
gcc::dump_manager::get_dump_file_name): Update prototype.
Index: dumpfile.c
===================================================================
--- dumpfile.c (revision 261885)
+++ dumpfile.c (working copy)
@@ -269,7 +269,7 @@ get_dump_file_info_by_switch (const char
char *
gcc::dump_manager::
-get_dump_file_name (int phase) const
+get_dump_file_name (int phase, int part) const
{
struct dump_file_info *dfi;
@@ -278,7 +278,7 @@ get_dump_file_name (int phase) const
dfi = get_dump_file_info (phase);
- return get_dump_file_name (dfi);
+ return get_dump_file_name (dfi, part);
}
/* Return the name of the dump file for the given dump_file_info.
@@ -288,7 +288,7 @@ get_dump_file_name (int phase) const
char *
gcc::dump_manager::
-get_dump_file_name (struct dump_file_info *dfi) const
+get_dump_file_name (struct dump_file_info *dfi, int part) const
{
char dump_id[10];
@@ -312,7 +312,14 @@ get_dump_file_name (struct dump_file_inf
dump_id[0] = '\0';
}
- return concat (dump_base_name, dump_id, dfi->suffix, NULL);
+ if (part != -1)
+ {
+ char part_id[8];
+ snprintf (part_id, sizeof (part_id), ".%i", part);
+ return concat (dump_base_name, dump_id, part_id, dfi->suffix, NULL);
+ }
+ else
+ return concat (dump_base_name, dump_id, dfi->suffix, NULL);
}
/* Open a dump file called FILENAME. Some filenames are special and
@@ -592,17 +599,19 @@ dump_finish (int phase)
/* Begin a tree dump for PHASE. Stores any user supplied flag in
*FLAG_PTR and returns a stream to write to. If the dump is not
enabled, returns NULL.
- Multiple calls will reopen and append to the dump file. */
+ PART can be used for dump files which should be split to multiple
+ parts. PART == -1 indicates dump file with no parts.
+ If PART is -1, multiple calls will reopen and append to the dump file. */
FILE *
-dump_begin (int phase, dump_flags_t *flag_ptr)
+dump_begin (int phase, dump_flags_t *flag_ptr, int part)
{
- return g->get_dumps ()->dump_begin (phase, flag_ptr);
+ return g->get_dumps ()->dump_begin (phase, flag_ptr, part);
}
FILE *
gcc::dump_manager::
-dump_begin (int phase, dump_flags_t *flag_ptr)
+dump_begin (int phase, dump_flags_t *flag_ptr, int part)
{
char *name;
struct dump_file_info *dfi;
@@ -611,12 +620,14 @@ dump_begin (int phase, dump_flags_t *fla
if (phase == TDI_none || !dump_phase_enabled_p (phase))
return NULL;
- name = get_dump_file_name (phase);
+ name = get_dump_file_name (phase, part);
if (!name)
return NULL;
dfi = get_dump_file_info (phase);
- stream = dump_open (name, dfi->pstate < 0);
+ /* We do not support re-opening of dump files with parts. This would require
+ tracking pstate per part of the dump file. */
+ stream = dump_open (name, part != -1 || dfi->pstate < 0);
if (stream)
dfi->pstate = 1;
free (name);
Index: dumpfile.h
===================================================================
--- dumpfile.h (revision 261885)
+++ dumpfile.h (working copy)
@@ -269,7 +269,7 @@ struct dump_file_info
};
/* In dumpfile.c */
-extern FILE *dump_begin (int, dump_flags_t *);
+extern FILE *dump_begin (int, dump_flags_t *, int part=-1);
extern void dump_end (int, FILE *);
extern int opt_info_switch_p (const char *);
extern const char *dump_flag_name (int);
@@ -343,10 +343,10 @@ public:
/* Return the name of the dump file for the given phase.
If the dump is not enabled, returns NULL. */
char *
- get_dump_file_name (int phase) const;
+ get_dump_file_name (int phase, int part = -1) const;
char *
- get_dump_file_name (struct dump_file_info *dfi) const;
+ get_dump_file_name (struct dump_file_info *dfi, int part = -1) const;
int
dump_switch_p (const char *arg);
@@ -365,7 +365,7 @@ public:
dump_finish (int phase);
FILE *
- dump_begin (int phase, dump_flags_t *flag_ptr);
+ dump_begin (int phase, dump_flags_t *flag_ptr, int part);
/* Returns nonzero if tree dump PHASE has been initialized. */
int
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: Add support for dumping multiple dump files under one name
2018-06-22 11:16 Add support for dumping multiple dump files under one name Jan Hubicka
@ 2018-06-29 8:47 ` Richard Biener
2018-06-29 11:17 ` David Malcolm
0 siblings, 1 reply; 6+ messages in thread
From: Richard Biener @ 2018-06-29 8:47 UTC (permalink / raw)
To: Jan Hubicka; +Cc: gcc-patches, dmalcolm
On Fri, 22 Jun 2018, Jan Hubicka wrote:
> Hi,
> this patch adds dumpfile support for dumps that come in multiple parts. This
> is needed for WPA stream-out dump since we stream partitions in parallel and
> the dumps would come up in random order. Parts are added by new parameter that
> is initialzed by default to -1 (no parts).
>
> One thing I skipped is any support for duplicated opening of file
> with parts since I do not need it.
>
> Bootstrapped/regtested x86_64-linux, OK?
Looks reasonable - David, anything you want to add / have changed?
Thanks,
Richard.
> Honza
>
> * dumpfile.c (gcc::dump_manager::get_dump_file_name): Add PART
> parameter.
> (gcc::dump_manager::get_dump_file_name): likewise.
> (dump_begin): Likewise.
> * dumpfile.h (dump_begin): Update prototype.
> (gcc::dump_manager::get_dump_file_name,
> gcc::dump_manager::get_dump_file_name): Update prototype.
> Index: dumpfile.c
> ===================================================================
> --- dumpfile.c (revision 261885)
> +++ dumpfile.c (working copy)
> @@ -269,7 +269,7 @@ get_dump_file_info_by_switch (const char
>
> char *
> gcc::dump_manager::
> -get_dump_file_name (int phase) const
> +get_dump_file_name (int phase, int part) const
> {
> struct dump_file_info *dfi;
>
> @@ -278,7 +278,7 @@ get_dump_file_name (int phase) const
>
> dfi = get_dump_file_info (phase);
>
> - return get_dump_file_name (dfi);
> + return get_dump_file_name (dfi, part);
> }
>
> /* Return the name of the dump file for the given dump_file_info.
> @@ -288,7 +288,7 @@ get_dump_file_name (int phase) const
>
> char *
> gcc::dump_manager::
> -get_dump_file_name (struct dump_file_info *dfi) const
> +get_dump_file_name (struct dump_file_info *dfi, int part) const
> {
> char dump_id[10];
>
> @@ -312,7 +312,14 @@ get_dump_file_name (struct dump_file_inf
> dump_id[0] = '\0';
> }
>
> - return concat (dump_base_name, dump_id, dfi->suffix, NULL);
> + if (part != -1)
> + {
> + char part_id[8];
> + snprintf (part_id, sizeof (part_id), ".%i", part);
> + return concat (dump_base_name, dump_id, part_id, dfi->suffix, NULL);
> + }
> + else
> + return concat (dump_base_name, dump_id, dfi->suffix, NULL);
> }
>
> /* Open a dump file called FILENAME. Some filenames are special and
> @@ -592,17 +599,19 @@ dump_finish (int phase)
> /* Begin a tree dump for PHASE. Stores any user supplied flag in
> *FLAG_PTR and returns a stream to write to. If the dump is not
> enabled, returns NULL.
> - Multiple calls will reopen and append to the dump file. */
> + PART can be used for dump files which should be split to multiple
> + parts. PART == -1 indicates dump file with no parts.
> + If PART is -1, multiple calls will reopen and append to the dump file. */
>
> FILE *
> -dump_begin (int phase, dump_flags_t *flag_ptr)
> +dump_begin (int phase, dump_flags_t *flag_ptr, int part)
> {
> - return g->get_dumps ()->dump_begin (phase, flag_ptr);
> + return g->get_dumps ()->dump_begin (phase, flag_ptr, part);
> }
>
> FILE *
> gcc::dump_manager::
> -dump_begin (int phase, dump_flags_t *flag_ptr)
> +dump_begin (int phase, dump_flags_t *flag_ptr, int part)
> {
> char *name;
> struct dump_file_info *dfi;
> @@ -611,12 +620,14 @@ dump_begin (int phase, dump_flags_t *fla
> if (phase == TDI_none || !dump_phase_enabled_p (phase))
> return NULL;
>
> - name = get_dump_file_name (phase);
> + name = get_dump_file_name (phase, part);
> if (!name)
> return NULL;
> dfi = get_dump_file_info (phase);
>
> - stream = dump_open (name, dfi->pstate < 0);
> + /* We do not support re-opening of dump files with parts. This would require
> + tracking pstate per part of the dump file. */
> + stream = dump_open (name, part != -1 || dfi->pstate < 0);
> if (stream)
> dfi->pstate = 1;
> free (name);
> Index: dumpfile.h
> ===================================================================
> --- dumpfile.h (revision 261885)
> +++ dumpfile.h (working copy)
> @@ -269,7 +269,7 @@ struct dump_file_info
> };
>
> /* In dumpfile.c */
> -extern FILE *dump_begin (int, dump_flags_t *);
> +extern FILE *dump_begin (int, dump_flags_t *, int part=-1);
> extern void dump_end (int, FILE *);
> extern int opt_info_switch_p (const char *);
> extern const char *dump_flag_name (int);
> @@ -343,10 +343,10 @@ public:
> /* Return the name of the dump file for the given phase.
> If the dump is not enabled, returns NULL. */
> char *
> - get_dump_file_name (int phase) const;
> + get_dump_file_name (int phase, int part = -1) const;
>
> char *
> - get_dump_file_name (struct dump_file_info *dfi) const;
> + get_dump_file_name (struct dump_file_info *dfi, int part = -1) const;
>
> int
> dump_switch_p (const char *arg);
> @@ -365,7 +365,7 @@ public:
> dump_finish (int phase);
>
> FILE *
> - dump_begin (int phase, dump_flags_t *flag_ptr);
> + dump_begin (int phase, dump_flags_t *flag_ptr, int part);
>
> /* Returns nonzero if tree dump PHASE has been initialized. */
> int
>
>
--
Richard Biener <rguenther@suse.de>
SUSE LINUX GmbH, GF: Felix Imendoerffer, Jane Smithard, Graham Norton, HRB 21284 (AG Nuernberg)
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: Add support for dumping multiple dump files under one name
2018-06-29 8:47 ` Richard Biener
@ 2018-06-29 11:17 ` David Malcolm
2018-07-03 10:01 ` Andre Vieira (lists)
0 siblings, 1 reply; 6+ messages in thread
From: David Malcolm @ 2018-06-29 11:17 UTC (permalink / raw)
To: Richard Biener, Jan Hubicka; +Cc: gcc-patches
On Fri, 2018-06-29 at 10:15 +0200, Richard Biener wrote:
> On Fri, 22 Jun 2018, Jan Hubicka wrote:
>
> > Hi,
> > this patch adds dumpfile support for dumps that come in multiple
> > parts. This
> > is needed for WPA stream-out dump since we stream partitions in
> > parallel and
> > the dumps would come up in random order. Parts are added by new
> > parameter that
> > is initialzed by default to -1 (no parts).
> >
> > One thing I skipped is any support for duplicated opening of file
> > with parts since I do not need it.
> >
> > Bootstrapped/regtested x86_64-linux, OK?
>
> Looks reasonable - David, anything you want to add / have changed?
No worries from my side; I don't think it interacts with the
optimization records stuff I'm working on - presumably this is just for
dumping the WPA stream-out, rather than for dumping specific
optimizations.
[...snip...]
Dave
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: Add support for dumping multiple dump files under one name
2018-06-29 11:17 ` David Malcolm
@ 2018-07-03 10:01 ` Andre Vieira (lists)
2018-07-03 14:15 ` David Malcolm
0 siblings, 1 reply; 6+ messages in thread
From: Andre Vieira (lists) @ 2018-07-03 10:01 UTC (permalink / raw)
To: David Malcolm; +Cc: gcc-patches
On 29/06/18 11:13, David Malcolm wrote:
> On Fri, 2018-06-29 at 10:15 +0200, Richard Biener wrote:
>> On Fri, 22 Jun 2018, Jan Hubicka wrote:
>>
>>> Hi,
>>> this patch adds dumpfile support for dumps that come in multiple
>>> parts. This
>>> is needed for WPA stream-out dump since we stream partitions in
>>> parallel and
>>> the dumps would come up in random order. Parts are added by new
>>> parameter that
>>> is initialzed by default to -1 (no parts).
>>>
>>> One thing I skipped is any support for duplicated opening of file
>>> with parts since I do not need it.
>>>
>>> Bootstrapped/regtested x86_64-linux, OK?
>>
>> Looks reasonable - David, anything you want to add / have changed?
>
> No worries from my side; I don't think it interacts with the
> optimization records stuff I'm working on - presumably this is just for
> dumping the WPA stream-out, rather than for dumping specific
> optimizations.
>
> [...snip...]
>
> Dave
>
Hi David,
I believe r262245 is causing the following failures on aarch64 and arm:
FAIL: gcc.dg/vect/slp-perm-1.c -flto -ffat-lto-objects scan-tree-dump
vect "note: Built SLP cancelled: can use load/store-lanes"
FAIL: gcc.dg/vect/slp-perm-1.c scan-tree-dump vect "note: Built SLP
cancelled: can use load/store-lanes"
FAIL: gcc.dg/vect/slp-perm-2.c -flto -ffat-lto-objects scan-tree-dump
vect "note: Built SLP cancelled: can use load/store-lanes"
FAIL: gcc.dg/vect/slp-perm-2.c scan-tree-dump vect "note: Built SLP
cancelled: can use load/store-lanes"
FAIL: gcc.dg/vect/slp-perm-3.c -flto -ffat-lto-objects scan-tree-dump
vect "note: Built SLP cancelled: can use load/store-lanes"
FAIL: gcc.dg/vect/slp-perm-3.c scan-tree-dump vect "note: Built SLP
cancelled: can use load/store-lanes"
FAIL: gcc.dg/vect/slp-perm-5.c -flto -ffat-lto-objects scan-tree-dump
vect "note: Built SLP cancelled: can use load/store-lanes"
FAIL: gcc.dg/vect/slp-perm-5.c scan-tree-dump vect "note: Built SLP
cancelled: can use load/store-lanes"
FAIL: gcc.dg/vect/slp-perm-6.c -flto -ffat-lto-objects scan-tree-dump
vect "note: Built SLP cancelled: can use load/store-lanes"
FAIL: gcc.dg/vect/slp-perm-6.c scan-tree-dump vect "note: Built SLP
cancelled: can use load/store-lanes"
FAIL: gcc.dg/vect/slp-perm-7.c -flto -ffat-lto-objects scan-tree-dump
vect "note: Built SLP cancelled: can use load/store-lanes"
FAIL: gcc.dg/vect/slp-perm-7.c scan-tree-dump vect "note: Built SLP
cancelled: can use load/store-lanes"
FAIL: gcc.dg/vect/slp-perm-8.c -flto -ffat-lto-objects scan-tree-dump
vect "note: Built SLP cancelled: can use load/store-lanes"
FAIL: gcc.dg/vect/slp-perm-8.c scan-tree-dump vect "note: Built SLP
cancelled: can use load/store-lanes"
Could you please have a look?
Cheers,
Andre
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: Add support for dumping multiple dump files under one name
2018-07-03 10:01 ` Andre Vieira (lists)
@ 2018-07-03 14:15 ` David Malcolm
2018-07-03 16:20 ` Andre Vieira (lists)
0 siblings, 1 reply; 6+ messages in thread
From: David Malcolm @ 2018-07-03 14:15 UTC (permalink / raw)
To: Andre Vieira (lists); +Cc: gcc-patches
On Tue, 2018-07-03 at 11:00 +0100, Andre Vieira (lists) wrote:
> On 29/06/18 11:13, David Malcolm wrote:
> > On Fri, 2018-06-29 at 10:15 +0200, Richard Biener wrote:
> > > On Fri, 22 Jun 2018, Jan Hubicka wrote:
> > >
> > > > Hi,
> > > > this patch adds dumpfile support for dumps that come in
> > > > multiple
> > > > parts. This
> > > > is needed for WPA stream-out dump since we stream partitions in
> > > > parallel and
> > > > the dumps would come up in random order. Parts are added by
> > > > new
> > > > parameter that
> > > > is initialzed by default to -1 (no parts).
> > > >
> > > > One thing I skipped is any support for duplicated opening of
> > > > file
> > > > with parts since I do not need it.
> > > >
> > > > Bootstrapped/regtested x86_64-linux, OK?
> > >
> > > Looks reasonable - David, anything you want to add / have
> > > changed?
> >
> > No worries from my side; I don't think it interacts with the
> > optimization records stuff I'm working on - presumably this is just
> > for
> > dumping the WPA stream-out, rather than for dumping specific
> > optimizations.
> >
> > [...snip...]
> >
> > Dave
> >
>
> Hi David,
>
> I believe r262245 is causing the following failures on aarch64 and
> arm:
>
> FAIL: gcc.dg/vect/slp-perm-1.c -flto -ffat-lto-objects scan-tree-
> dump
> vect "note: Built SLP cancelled: can use load/store-lanes"
> FAIL: gcc.dg/vect/slp-perm-1.c scan-tree-dump vect "note: Built SLP
> cancelled: can use load/store-lanes"
> FAIL: gcc.dg/vect/slp-perm-2.c -flto -ffat-lto-objects scan-tree-
> dump
> vect "note: Built SLP cancelled: can use load/store-lanes"
> FAIL: gcc.dg/vect/slp-perm-2.c scan-tree-dump vect "note: Built SLP
> cancelled: can use load/store-lanes"
> FAIL: gcc.dg/vect/slp-perm-3.c -flto -ffat-lto-objects scan-tree-
> dump
> vect "note: Built SLP cancelled: can use load/store-lanes"
> FAIL: gcc.dg/vect/slp-perm-3.c scan-tree-dump vect "note: Built SLP
> cancelled: can use load/store-lanes"
> FAIL: gcc.dg/vect/slp-perm-5.c -flto -ffat-lto-objects scan-tree-
> dump
> vect "note: Built SLP cancelled: can use load/store-lanes"
> FAIL: gcc.dg/vect/slp-perm-5.c scan-tree-dump vect "note: Built SLP
> cancelled: can use load/store-lanes"
> FAIL: gcc.dg/vect/slp-perm-6.c -flto -ffat-lto-objects scan-tree-
> dump
> vect "note: Built SLP cancelled: can use load/store-lanes"
> FAIL: gcc.dg/vect/slp-perm-6.c scan-tree-dump vect "note: Built SLP
> cancelled: can use load/store-lanes"
> FAIL: gcc.dg/vect/slp-perm-7.c -flto -ffat-lto-objects scan-tree-
> dump
> vect "note: Built SLP cancelled: can use load/store-lanes"
> FAIL: gcc.dg/vect/slp-perm-7.c scan-tree-dump vect "note: Built SLP
> cancelled: can use load/store-lanes"
> FAIL: gcc.dg/vect/slp-perm-8.c -flto -ffat-lto-objects scan-tree-
> dump
> vect "note: Built SLP cancelled: can use load/store-lanes"
> FAIL: gcc.dg/vect/slp-perm-8.c scan-tree-dump vect "note: Built SLP
> cancelled: can use load/store-lanes"
>
> Could you please have a look?
Sorry about this; I think my r262246 ("dumpfile.c: add indentation via
DUMP_VECT_SCOPE") caused this.
Does https://gcc.gnu.org/ml/gcc-patches/2018-07/msg00122.html help?
Dave
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: Add support for dumping multiple dump files under one name
2018-07-03 14:15 ` David Malcolm
@ 2018-07-03 16:20 ` Andre Vieira (lists)
0 siblings, 0 replies; 6+ messages in thread
From: Andre Vieira (lists) @ 2018-07-03 16:20 UTC (permalink / raw)
To: David Malcolm; +Cc: gcc-patches
On 03/07/18 15:15, David Malcolm wrote:
> On Tue, 2018-07-03 at 11:00 +0100, Andre Vieira (lists) wrote:
>> On 29/06/18 11:13, David Malcolm wrote:
>>> On Fri, 2018-06-29 at 10:15 +0200, Richard Biener wrote:
>>>> On Fri, 22 Jun 2018, Jan Hubicka wrote:
>>>>
>>>>> Hi,
>>>>> this patch adds dumpfile support for dumps that come in
>>>>> multiple
>>>>> parts. This
>>>>> is needed for WPA stream-out dump since we stream partitions in
>>>>> parallel and
>>>>> the dumps would come up in random order. Parts are added by
>>>>> new
>>>>> parameter that
>>>>> is initialzed by default to -1 (no parts).
>>>>>
>>>>> One thing I skipped is any support for duplicated opening of
>>>>> file
>>>>> with parts since I do not need it.
>>>>>
>>>>> Bootstrapped/regtested x86_64-linux, OK?
>>>>
>>>> Looks reasonable - David, anything you want to add / have
>>>> changed?
>>>
>>> No worries from my side; I don't think it interacts with the
>>> optimization records stuff I'm working on - presumably this is just
>>> for
>>> dumping the WPA stream-out, rather than for dumping specific
>>> optimizations.
>>>
>>> [...snip...]
>>>
>>> Dave
>>>
>>
>> Hi David,
>>
>> I believe r262245 is causing the following failures on aarch64 and
>> arm:
>>
>> FAIL: gcc.dg/vect/slp-perm-1.c -flto -ffat-lto-objects scan-tree-
>> dump
>> vect "note: Built SLP cancelled: can use load/store-lanes"
>> FAIL: gcc.dg/vect/slp-perm-1.c scan-tree-dump vect "note: Built SLP
>> cancelled: can use load/store-lanes"
>> FAIL: gcc.dg/vect/slp-perm-2.c -flto -ffat-lto-objects scan-tree-
>> dump
>> vect "note: Built SLP cancelled: can use load/store-lanes"
>> FAIL: gcc.dg/vect/slp-perm-2.c scan-tree-dump vect "note: Built SLP
>> cancelled: can use load/store-lanes"
>> FAIL: gcc.dg/vect/slp-perm-3.c -flto -ffat-lto-objects scan-tree-
>> dump
>> vect "note: Built SLP cancelled: can use load/store-lanes"
>> FAIL: gcc.dg/vect/slp-perm-3.c scan-tree-dump vect "note: Built SLP
>> cancelled: can use load/store-lanes"
>> FAIL: gcc.dg/vect/slp-perm-5.c -flto -ffat-lto-objects scan-tree-
>> dump
>> vect "note: Built SLP cancelled: can use load/store-lanes"
>> FAIL: gcc.dg/vect/slp-perm-5.c scan-tree-dump vect "note: Built SLP
>> cancelled: can use load/store-lanes"
>> FAIL: gcc.dg/vect/slp-perm-6.c -flto -ffat-lto-objects scan-tree-
>> dump
>> vect "note: Built SLP cancelled: can use load/store-lanes"
>> FAIL: gcc.dg/vect/slp-perm-6.c scan-tree-dump vect "note: Built SLP
>> cancelled: can use load/store-lanes"
>> FAIL: gcc.dg/vect/slp-perm-7.c -flto -ffat-lto-objects scan-tree-
>> dump
>> vect "note: Built SLP cancelled: can use load/store-lanes"
>> FAIL: gcc.dg/vect/slp-perm-7.c scan-tree-dump vect "note: Built SLP
>> cancelled: can use load/store-lanes"
>> FAIL: gcc.dg/vect/slp-perm-8.c -flto -ffat-lto-objects scan-tree-
>> dump
>> vect "note: Built SLP cancelled: can use load/store-lanes"
>> FAIL: gcc.dg/vect/slp-perm-8.c scan-tree-dump vect "note: Built SLP
>> cancelled: can use load/store-lanes"
>>
>> Could you please have a look?
>
> Sorry about this; I think my r262246 ("dumpfile.c: add indentation via
> DUMP_VECT_SCOPE") caused this.
>
> Does https://gcc.gnu.org/ml/gcc-patches/2018-07/msg00122.html help?
Yes it does. Thank you!
Andre
>
> Dave
>
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2018-07-03 16:20 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-06-22 11:16 Add support for dumping multiple dump files under one name Jan Hubicka
2018-06-29 8:47 ` Richard Biener
2018-06-29 11:17 ` David Malcolm
2018-07-03 10:01 ` Andre Vieira (lists)
2018-07-03 14:15 ` David Malcolm
2018-07-03 16:20 ` Andre Vieira (lists)
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).