Re: [PATCH] Fix reassoc qsort checking ICE (PR tree-optimization/83221)

2017-11-30 Thread Richard Biener
On December 1, 2017 12:10:45 AM GMT+01:00, Jakub Jelinek wrote: >Hi! > >bb_rank is long and has basic block indexes << 16, and oe rank >is unsigned int. > >So, if some function has over 32767 basic blocks, we can run into >various >issues. > >As I said in the PR, I see 3 possible fixes, one is at

[PATCH] Fix reassoc qsort checking ICE (PR tree-optimization/83221)

2017-11-30 Thread Jakub Jelinek
Hi! bb_rank is long and has basic block indexes << 16, and oe rank is unsigned int. So, if some function has over 32767 basic blocks, we can run into various issues. As I said in the PR, I see 3 possible fixes, one is attached below and the shortest, which I've bootstrapped/regtested on x86_64-l