On Wed, Jan 14, 2015 at 04:28:42PM +0300, Yuri Rumyantsev wrote:
> Hi All,
>
> I did all changes proposed by Richard and delete check on def in the
> same block as Jakub proposed.
> I also moved check on optimization to call site..
>
> I also checked that bootstrap and regression testing did not show any
> new failures.
>
> Is it OK for trunk?
The | SSA_OP_VUSE is still in there, the testcase is still executable,
still doesn't end with newline, and I really think you should replace
lattice = XALLOCAVEC (unsigned int, n);
with something like:
if (n >= 100000)
lattice = XNEWVEC (unsigned int, n);
else
lattice = XALLOCAVEC (unsigned int, n);
...
if (n >= 100000)
XDELETE (lattice);
or similar.
Jakub
