Re: [PATCH][store-merging] Use store order as tie-breaker in sort_by_bitpos

2017-09-13 Thread Richard Biener
On Wed, Sep 13, 2017 at 3:02 PM, Kyrill Tkachov wrote: > Hi all, > > As Alexander pointed out in the thread starting at [1] the sort_by_bitpos > sorting function > was behaving badly when we had multiple stores at the same position. He > fixed that (thanks!) > but

[PATCH][store-merging] Use store order as tie-breaker in sort_by_bitpos

2017-09-13 Thread Kyrill Tkachov
Hi all, As Alexander pointed out in the thread starting at [1] the sort_by_bitpos sorting function was behaving badly when we had multiple stores at the same position. He fixed that (thanks!) but we can do better by not returning zero when the bitpositions are equal but by falling back to