2014-10-09 22:51 GMT+04:00 Jeff Law <l...@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.enkov...@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. Ilya
