On Thu, Oct 4, 2012 at 5:37 PM, Vladimir Makarov <vmaka...@redhat.com> wrote: > The only issue now is PR54146 compilation time for IRA+LRA although it > was improved significantly. I will continue work on PR54146. But now I > am going to focus on proposals from reviews.
Right, there still are opportunities to improve things. (The real solution may be to stop SRA from creating so many simultaneously live pseudos in the first place...) > + lra_simple_p > + = (ira_use_lra_p && max_reg_num () >= (1 << 26) / last_basic_block); I think you should use n_basic_blocks here instead of last_basic_block, in case this runs without compacting the cfg first (n_basic_blocks is the real number of basic blocks in the cfg, last_basic_block is the highest index, so last_basic_block >= n_basic_blocks). Thanks for working on this! Ciao! Steven
