Re: [PATCH] gimple UIDs, LTO and -fanalyzer [PR98599]

2021-01-21 Thread David Malcolm via Gcc-patches
On Thu, 2021-01-21 at 20:09 +0100, Jan Hubicka wrote: > > On Thu, 2021-01-14 at 15:00 +0100, Jan Hubicka wrote: > > > > On Wed, Jan 13, 2021 at 11:04 PM David Malcolm via Gcc-patches > > > > wrote: > > > > > gimple.h has this comment for gimple's uid field: > > > > > > > > > > /* UID of this st

Re: [PATCH] gimple UIDs, LTO and -fanalyzer [PR98599]

2021-01-21 Thread David Malcolm via Gcc-patches
On Thu, 2021-01-21 at 20:09 +0100, Jan Hubicka wrote: > > On Thu, 2021-01-14 at 15:00 +0100, Jan Hubicka wrote: > > > > On Wed, Jan 13, 2021 at 11:04 PM David Malcolm via Gcc-patches > > > > wrote: > > > > > gimple.h has this comment for gimple's uid field: > > > > > > > > > > /* UID of this st

Re: [PATCH] gimple UIDs, LTO and -fanalyzer [PR98599]

2021-01-21 Thread Jan Hubicka
> On Thu, 2021-01-14 at 15:00 +0100, Jan Hubicka wrote: > > > On Wed, Jan 13, 2021 at 11:04 PM David Malcolm via Gcc-patches > > > wrote: > > > > gimple.h has this comment for gimple's uid field: > > > > > > > > /* UID of this statement. This is used by passes that want to > > > > assign

Re: [PATCH] gimple UIDs, LTO and -fanalyzer [PR98599]

2021-01-21 Thread David Malcolm via Gcc-patches
On Thu, 2021-01-14 at 15:00 +0100, Jan Hubicka wrote: > > On Wed, Jan 13, 2021 at 11:04 PM David Malcolm via Gcc-patches > > wrote: > > > gimple.h has this comment for gimple's uid field: > > > > > > /* UID of this statement. This is used by passes that want to > > > assign IDs to stateme

Re: [PATCH] gimple UIDs, LTO and -fanalyzer [PR98599]

2021-01-14 Thread Jan Hubicka
> On Wed, Jan 13, 2021 at 11:04 PM David Malcolm via Gcc-patches > wrote: > > > > gimple.h has this comment for gimple's uid field: > > > > /* UID of this statement. This is used by passes that want to > > assign IDs to statements. It must be assigned and used by each > > pass. By d

Re: [PATCH] gimple UIDs, LTO and -fanalyzer [PR98599]

2021-01-13 Thread Richard Biener via Gcc-patches
On Wed, Jan 13, 2021 at 11:04 PM David Malcolm via Gcc-patches wrote: > > gimple.h has this comment for gimple's uid field: > > /* UID of this statement. This is used by passes that want to > assign IDs to statements. It must be assigned and used by each > pass. By default it should