Re: [PATCH 35/41] analyzer: new file: exploded-graph.h

2020-01-10 Thread Jeff Law
On Fri, 2020-01-10 at 12:18 -0500, David Malcolm wrote: > On Fri, 2020-01-10 at 09:10 -0700, Jeff Law wrote: > > On Wed, 2020-01-08 at 04:02 -0500, David Malcolm wrote: > > > Jeff's initial review of v1 of this patch: > > > https://gcc.gnu.org/ml/gcc-patches/2019-12/msg00813.html > > > I've addre

Re: [PATCH 35/41] analyzer: new file: exploded-graph.h

2020-01-10 Thread David Malcolm
On Fri, 2020-01-10 at 09:10 -0700, Jeff Law wrote: > On Wed, 2020-01-08 at 04:02 -0500, David Malcolm wrote: > > Jeff's initial review of v1 of this patch: > > https://gcc.gnu.org/ml/gcc-patches/2019-12/msg00813.html > > I've addressed most of the issues he raised there. > > TODO: should some str

Re: [PATCH 35/41] analyzer: new file: exploded-graph.h

2020-01-10 Thread Jeff Law
On Wed, 2020-01-08 at 04:02 -0500, David Malcolm wrote: > Jeff's initial review of v1 of this patch: > https://gcc.gnu.org/ml/gcc-patches/2019-12/msg00813.html > I've addressed most of the issues he raised there. > TODO: should some structs be classes? > > Changed in v5: > - update ChangeLog pat