[PATCH PR64434]

Jakub Jelinek jakub@redhat.com
Wed Jan 14 13:49:00 GMT 2015


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



More information about the Gcc-patches mailing list