Re: [PATCH, Pointer Bounds Checker 14/x] Passes [10/n] Stores handler

[Jeff Law <law at redhat dot com>]

On 10/13/14 05:23, Ilya Enkovich wrote:
> 2014-10-09 22:51 GMT+04:00 Jeff Law <law@redhat.com>:
> > On 10/08/14 13:12, Ilya Enkovich wrote:
> > > Hi,
> > >
> > > This patch adds an assignment processing function which is used by lnliner
> > > for newly generated stores.
> > >
> > > Thanks,
> > > Ilya
> > > --
> > > 2014-10-08  Ilya Enkovich  <ilya.enkovich@intel.com>
> > >
> > >          * tree-chkp.c (chkp_copy_bounds_for_assign): New.
> > >          * tree-chkp.h (chkp_copy_bounds_for_assign): New.
> >
> > This probably should have been part of the inliner submission since that's
> > the only place its used and one needs the inliner context to know how this
> > function is going to be used.
> >
> > Presumably the reason its not in tree-inline and static is you want to
> > utilize chkp_walk_pointer_assignments?
> >
> > The code is fine, just want to make sure its goes into a logical place.
> >
> > Jeff
>
> I have to export either chkp_copy_bounds_for_assign or
> chkp_walk_pointer_assignments with chkp_copy_bounds_for_elem.  No much
> difference but I'd prefer to keep all memrefs processing codes in
> tree-chkp.c.
OK.
jeff