Re: [PATCH] cfgexpand: Update partition size when merging variables

2020-01-22 Thread Richard Biener
On Wed, Jan 22, 2020 at 11:59 AM Richard Sandiford wrote: > > cfgexpand sorts variables by decreasing size, so when merging a later > variable into an earlier one, there's usually no need to update the > merged size. > > But for poly_int sizes, the sort function just uses a lexicographical > compa

[PATCH] cfgexpand: Update partition size when merging variables

2020-01-22 Thread Richard Sandiford
cfgexpand sorts variables by decreasing size, so when merging a later variable into an earlier one, there's usually no need to update the merged size. But for poly_int sizes, the sort function just uses a lexicographical comparison of the coefficients, so e.g. 2X+2 comes before 0X+32. Which is big