On 11/20/2017 06:00 AM, Richard Sandiford wrote: > Jeff Law <l...@redhat.com> writes: >> On 10/23/2017 11:05 AM, Richard Sandiford wrote: >>> This patch makes indirect_refs_may_alias_p use ranges_may_overlap_p >>> rather than ranges_overlap_p. Unlike the former, the latter can handle >>> negative offsets, so the fix for PR44852 should no longer be necessary. >>> It can also handle offset_int, so avoids unchecked truncations to >>> HOST_WIDE_INT. >>> >>> >>> 2017-10-23 Richard Sandiford <richard.sandif...@linaro.org> >>> Alan Hayward <alan.hayw...@arm.com> >>> David Sherwood <david.sherw...@arm.com> >>> >>> gcc/ >>> * tree-ssa-alias.c (indirect_ref_may_alias_decl_p) >>> (indirect_refs_may_alias_p): Use ranges_may_overlap_p >>> instead of ranges_overlap_p. >> OK. >> >> Note that this highlighted a nit in patch 001 -- namely that there's new >> function templates that aren't mentioned in the ChangeLog. > > Do you mean ranges_may_overlap_p? I can add that and the other new > poly-int.h functions to the changelog if you think it's useful, > but I thought for new files it was more usual just to do: > > * foo.h: New file. That's fine. I was just having trouble finding it when I wanted to look at it. My mailer won't unwrap a compressed patch :-)
Jeff
