Re: Bug in tree-inline.c:estimate_num_insns_1?

[Andrew Pinski]

On Feb 24, 2005, at 10:07 AM, Richard Guenther wrote:

Hi!

In estimate_num_insns_1 we currently have:

    /* Recognize assignments of large structures and constructors of
       big arrays.  */
    case INIT_EXPR:
    case MODIFY_EXPR:
      x = TREE_OPERAND (x, 0);
      /* FALLTHRU */
    case TARGET_EXPR:
    case CONSTRUCTOR:
      {
        HOST_WIDE_INT size;
        ...

shouldn't TARGET_EXPR being moved up before x = TREE_OPERAND (x, 0); ?

TARGET_EXPR is not in gimple at all so really does not matter.

-- Pinski