ree.c 37.3 KB
Newer Older
1
/* Redundant Extension Elimination pass for the GNU compiler.
2
   Copyright (C) 2010-2014 Free Software Foundation, Inc.
3
   Contributed by Ilya Enkovich (ilya.enkovich@intel.com)
4

5 6
   Based on the Redundant Zero-extension elimination pass contributed by
   Sriraman Tallam (tmsriram@google.com) and Silvius Rus (rus@google.com).
7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26

This file is part of GCC.

GCC is free software; you can redistribute it and/or modify it under
the terms of the GNU General Public License as published by the Free
Software Foundation; either version 3, or (at your option) any later
version.

GCC is distributed in the hope that it will be useful, but WITHOUT ANY
WARRANTY; without even the implied warranty of MERCHANTABILITY or
FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
for more details.

You should have received a copy of the GNU General Public License
along with GCC; see the file COPYING3.  If not see
<http://www.gnu.org/licenses/>.  */


/* Problem Description :
   --------------------
27 28 29 30 31 32 33 34
   This pass is intended to remove redundant extension instructions.
   Such instructions appear for different reasons.  We expect some of
   them due to implicit zero-extension in 64-bit registers after writing
   to their lower 32-bit half (e.g. for the x86-64 architecture).
   Another possible reason is a type cast which follows a load (for
   instance a register restore) and which can be combined into a single
   instruction, and for which earlier local passes, e.g. the combiner,
   weren't able to optimize.
35 36 37 38 39

   How does this pass work  ?
   --------------------------

   This pass is run after register allocation.  Hence, all registers that
40 41 42 43 44
   this pass deals with are hard registers.  This pass first looks for an
   extension instruction that could possibly be redundant.  Such extension
   instructions show up in RTL with the pattern  :
   (set (reg:<SWI248> x) (any_extend:<SWI248> (reg:<SWI124> x))),
   where x can be any hard register.
45
   Now, this pass tries to eliminate this instruction by merging the
46
   extension with the definitions of register x.  For instance, if
47 48
   one of the definitions of register x was  :
   (set (reg:SI x) (plus:SI (reg:SI z1) (reg:SI z2))),
49 50
   followed by extension  :
   (set (reg:DI x) (zero_extend:DI (reg:SI x)))
51 52 53
   then the combination converts this into :
   (set (reg:DI x) (zero_extend:DI (plus:SI (reg:SI z1) (reg:SI z2)))).
   If all the merged definitions are recognizable assembly instructions,
54 55 56 57 58 59 60
   the extension is effectively eliminated.

   For example, for the x86-64 architecture, implicit zero-extensions
   are captured with appropriate patterns in the i386.md file.  Hence,
   these merged definition can be matched to a single assembly instruction.
   The original extension instruction is then deleted if all the
   definitions can be merged.
61 62

   However, there are cases where the definition instruction cannot be
63 64
   merged with an extension.  Examples are CALL instructions.  In such
   cases, the original extension is not redundant and this pass does
65 66 67 68 69
   not delete it.

   Handling conditional moves :
   ----------------------------

70 71
   Architectures like x86-64 support conditional moves whose semantics for
   extension differ from the other instructions.  For instance, the
72 73
   instruction *cmov ebx, eax*
   zero-extends eax onto rax only when the move from ebx to eax happens.
74
   Otherwise, eax may not be zero-extended.  Consider conditional moves as
75 76
   RTL instructions of the form
   (set (reg:SI x) (if_then_else (cond) (reg:SI y) (reg:SI z))).
77
   This pass tries to merge an extension with a conditional move by
78
   actually merging the definitions of y and z with an extension and then
79 80
   converting the conditional move into :
   (set (reg:DI x) (if_then_else (cond) (reg:DI y) (reg:DI z))).
81 82 83
   Since registers y and z are extended, register x will also be extended
   after the conditional move.  Note that this step has to be done
   transitively since the definition of a conditional copy can be
84 85 86 87 88 89 90 91 92 93 94 95 96 97 98 99 100 101 102 103
   another conditional copy.

   Motivating Example I :
   ---------------------
   For this program :
   **********************************************
   bad_code.c

   int mask[1000];

   int foo(unsigned x)
   {
     if (x < 10)
       x = x * 45;
     else
       x = x * 78;
     return mask[x];
   }
   **********************************************

104
   $ gcc -O2 bad_code.c
105 106 107
     ........
     400315:       b8 4e 00 00 00          mov    $0x4e,%eax
     40031a:       0f af f8                imul   %eax,%edi
108
     40031d:       89 ff                   mov    %edi,%edi - useless extension
109 110 111 112 113
     40031f:       8b 04 bd 60 19 40 00    mov    0x401960(,%rdi,4),%eax
     400326:       c3                      retq
     ......
     400330:       ba 2d 00 00 00          mov    $0x2d,%edx
     400335:       0f af fa                imul   %edx,%edi
114
     400338:       89 ff                   mov    %edi,%edi - useless extension
115 116 117
     40033a:       8b 04 bd 60 19 40 00    mov    0x401960(,%rdi,4),%eax
     400341:       c3                      retq

118
   $ gcc -O2 -free bad_code.c
119 120 121 122 123 124 125 126 127 128 129 130 131 132 133 134 135 136 137 138 139 140 141 142 143 144
     ......
     400315:       6b ff 4e                imul   $0x4e,%edi,%edi
     400318:       8b 04 bd 40 19 40 00    mov    0x401940(,%rdi,4),%eax
     40031f:       c3                      retq
     400320:       6b ff 2d                imul   $0x2d,%edi,%edi
     400323:       8b 04 bd 40 19 40 00    mov    0x401940(,%rdi,4),%eax
     40032a:       c3                      retq

   Motivating Example II :
   ---------------------

   Here is an example with a conditional move.

   For this program :
   **********************************************

   unsigned long long foo(unsigned x , unsigned y)
   {
     unsigned z;
     if (x > 100)
       z = x + y;
     else
       z = x - y;
     return (unsigned long long)(z);
   }

145
   $ gcc -O2 bad_code.c
146 147 148 149 150 151
     ............
     400360:       8d 14 3e                lea    (%rsi,%rdi,1),%edx
     400363:       89 f8                   mov    %edi,%eax
     400365:       29 f0                   sub    %esi,%eax
     400367:       83 ff 65                cmp    $0x65,%edi
     40036a:       0f 43 c2                cmovae %edx,%eax
152
     40036d:       89 c0                   mov    %eax,%eax - useless extension
153 154
     40036f:       c3                      retq

155
   $ gcc -O2 -free bad_code.c
156 157 158 159 160 161 162 163 164
     .............
     400360:       89 fa                   mov    %edi,%edx
     400362:       8d 04 3e                lea    (%rsi,%rdi,1),%eax
     400365:       29 f2                   sub    %esi,%edx
     400367:       83 ff 65                cmp    $0x65,%edi
     40036a:       89 d6                   mov    %edx,%esi
     40036c:       48 0f 42 c6             cmovb  %rsi,%rax
     400370:       c3                      retq

165 166 167 168 169 170 171 172 173 174 175 176 177 178 179 180 181 182 183 184 185 186 187 188 189 190
  Motivating Example III :
  ---------------------

  Here is an example with a type cast.

  For this program :
  **********************************************

  void test(int size, unsigned char *in, unsigned char *out)
  {
    int i;
    unsigned char xr, xg, xy=0;

    for (i = 0; i < size; i++) {
      xr = *in++;
      xg = *in++;
      xy = (unsigned char) ((19595*xr + 38470*xg) >> 16);
      *out++ = xy;
    }
  }

  $ gcc -O2 bad_code.c
    ............
    10:   0f b6 0e                movzbl (%rsi),%ecx
    13:   0f b6 46 01             movzbl 0x1(%rsi),%eax
    17:   48 83 c6 02             add    $0x2,%rsi
191 192
    1b:   0f b6 c9                movzbl %cl,%ecx - useless extension
    1e:   0f b6 c0                movzbl %al,%eax - useless extension
193 194 195 196 197 198 199 200 201 202
    21:   69 c9 8b 4c 00 00       imul   $0x4c8b,%ecx,%ecx
    27:   69 c0 46 96 00 00       imul   $0x9646,%eax,%eax

   $ gcc -O2 -free bad_code.c
     .............
    10:   0f b6 0e                movzbl (%rsi),%ecx
    13:   0f b6 46 01             movzbl 0x1(%rsi),%eax
    17:   48 83 c6 02             add    $0x2,%rsi
    1b:   69 c9 8b 4c 00 00       imul   $0x4c8b,%ecx,%ecx
    21:   69 c0 46 96 00 00       imul   $0x9646,%eax,%eax
203 204 205 206

   Usefulness :
   ----------

207 208 209
   The original redundant zero-extension elimination pass reported reduction
   of the dynamic instruction count of a compression benchmark by 2.8% and
   improvement of its run time by about 1%.
210

211 212 213 214 215
   The additional performance gain with the enhanced pass is mostly expected
   on in-order architectures where redundancy cannot be compensated by out of
   order execution.  Measurements showed up to 10% performance gain (reduced
   run time) on EEMBC 2.0 benchmarks on Atom processor with geomean performance
   gain 1%.  */
216 217 218 219 220 221 222 223 224 225 226 227 228 229


#include "config.h"
#include "system.h"
#include "coretypes.h"
#include "tm.h"
#include "rtl.h"
#include "tree.h"
#include "tm_p.h"
#include "flags.h"
#include "regs.h"
#include "hard-reg-set.h"
#include "basic-block.h"
#include "insn-config.h"
230 231 232 233 234
#include "hashtab.h"
#include "hash-set.h"
#include "vec.h"
#include "machmode.h"
#include "input.h"
235 236 237 238
#include "function.h"
#include "expr.h"
#include "insn-attr.h"
#include "recog.h"
239
#include "diagnostic-core.h"
240 241 242 243 244 245 246 247 248
#include "target.h"
#include "optabs.h"
#include "insn-codes.h"
#include "rtlhooks-def.h"
#include "params.h"
#include "tree-pass.h"
#include "df.h"
#include "cgraph.h"

249
/* This structure represents a candidate for elimination.  */
250

251
typedef struct ext_cand
252
{
253 254
  /* The expression.  */
  const_rtx expr;
255

256 257
  /* The kind of extension.  */
  enum rtx_code code;
258

259 260 261 262
  /* The destination mode.  */
  enum machine_mode mode;

  /* The instruction where it lives.  */
David Malcolm committed
263
  rtx_insn *insn;
264
} ext_cand;
265 266


267 268
static int max_insn_uid;

269 270 271 272 273 274
/* Given a insn (CURR_INSN), an extension candidate for removal (CAND)
   and a pointer to the SET rtx (ORIG_SET) that needs to be modified,
   this code modifies the SET rtx to a new SET rtx that extends the
   right hand expression into a register on the left hand side.  Note
   that multiple assumptions are made about the nature of the set that
   needs to be true for this to work and is called from merge_def_and_ext.
275 276

   Original :
277
   (set (reg a) (expression))
278 279

   Transform :
280
   (set (reg a) (any_extend (expression)))
281 282

   Special Cases :
283
   If the expression is a constant or another extension, then directly
284
   assign it to the register.  */
285 286

static bool
David Malcolm committed
287
combine_set_extension (ext_cand *cand, rtx_insn *curr_insn, rtx *orig_set)
288
{
289 290
  rtx orig_src = SET_SRC (*orig_set);
  rtx new_set;
291 292 293 294 295 296 297 298 299 300 301 302 303
  rtx cand_pat = PATTERN (cand->insn);

  /* If the extension's source/destination registers are not the same
     then we need to change the original load to reference the destination
     of the extension.  Then we need to emit a copy from that destination
     to the original destination of the load.  */
  rtx new_reg;
  bool copy_needed
    = (REGNO (SET_DEST (cand_pat)) != REGNO (XEXP (SET_SRC (cand_pat), 0)));
  if (copy_needed)
    new_reg = gen_rtx_REG (cand->mode, REGNO (SET_DEST (cand_pat)));
  else
    new_reg = gen_rtx_REG (cand->mode, REGNO (SET_DEST (*orig_set)));
304

305 306
#if 0
  /* Rethinking test.  Temporarily disabled.  */
307 308 309
  /* We're going to be widening the result of DEF_INSN, ensure that doing so
     doesn't change the number of hard registers needed for the result.  */
  if (HARD_REGNO_NREGS (REGNO (new_reg), cand->mode)
310 311
      != HARD_REGNO_NREGS (REGNO (SET_DEST (*orig_set)),
			   GET_MODE (SET_DEST (*orig_set))))
312
	return false;
313
#endif
314

315 316
  /* Merge constants by directly moving the constant into the register under
     some conditions.  Recall that RTL constants are sign-extended.  */
317
  if (GET_CODE (orig_src) == CONST_INT
318
      && HOST_BITS_PER_WIDE_INT >= GET_MODE_BITSIZE (cand->mode))
319
    {
320 321
      if (INTVAL (orig_src) >= 0 || cand->code == SIGN_EXTEND)
	new_set = gen_rtx_SET (VOIDmode, new_reg, orig_src);
322
      else
323 324 325 326
	{
	  /* Zero-extend the negative constant by masking out the bits outside
	     the source mode.  */
	  enum machine_mode src_mode = GET_MODE (SET_DEST (*orig_set));
327
	  rtx new_const_int
328 329
	    = gen_int_mode (INTVAL (orig_src) & GET_MODE_MASK (src_mode),
			    GET_MODE (new_reg));
330
	  new_set = gen_rtx_SET (VOIDmode, new_reg, new_const_int);
331 332 333 334
	}
    }
  else if (GET_MODE (orig_src) == VOIDmode)
    {
335
      /* This is mostly due to a call insn that should not be optimized.  */
336
      return false;
337
    }
338
  else if (GET_CODE (orig_src) == cand->code)
339
    {
340 341 342 343
      /* Here is a sequence of two extensions.  Try to merge them.  */
      rtx temp_extension
	= gen_rtx_fmt_e (cand->code, cand->mode, XEXP (orig_src, 0));
      rtx simplified_temp_extension = simplify_rtx (temp_extension);
344 345
      if (simplified_temp_extension)
        temp_extension = simplified_temp_extension;
346
      new_set = gen_rtx_SET (VOIDmode, new_reg, temp_extension);
347 348 349
    }
  else if (GET_CODE (orig_src) == IF_THEN_ELSE)
    {
350
      /* Only IF_THEN_ELSE of phi-type copies are combined.  Otherwise,
351 352 353 354 355
         in general, IF_THEN_ELSE should not be combined.  */
      return false;
    }
  else
    {
356 357 358 359
      /* This is the normal case.  */
      rtx temp_extension
	= gen_rtx_fmt_e (cand->code, cand->mode, orig_src);
      rtx simplified_temp_extension = simplify_rtx (temp_extension);
360 361
      if (simplified_temp_extension)
        temp_extension = simplified_temp_extension;
362
      new_set = gen_rtx_SET (VOIDmode, new_reg, temp_extension);
363 364
    }

365
  /* This change is a part of a group of changes.  Hence,
366 367 368 369 370
     validate_change will not try to commit the change.  */
  if (validate_change (curr_insn, orig_set, new_set, true))
    {
      if (dump_file)
        {
371
          fprintf (dump_file,
372 373
		   "Tentatively merged extension with definition %s:\n",
		   (copy_needed) ? "(copy needed)" : "");
374 375 376 377
          print_rtl_single (dump_file, curr_insn);
        }
      return true;
    }
378

379 380 381 382
  return false;
}

/* Treat if_then_else insns, where the operands of both branches
383
   are registers, as copies.  For instance,
384 385 386 387 388 389 390
   Original :
   (set (reg:SI a) (if_then_else (cond) (reg:SI b) (reg:SI c)))
   Transformed :
   (set (reg:DI a) (if_then_else (cond) (reg:DI b) (reg:DI c)))
   DEF_INSN is the if_then_else insn.  */

static bool
David Malcolm committed
391
transform_ifelse (ext_cand *cand, rtx_insn *def_insn)
392 393 394 395 396 397 398 399 400
{
  rtx set_insn = PATTERN (def_insn);
  rtx srcreg, dstreg, srcreg2;
  rtx map_srcreg, map_dstreg, map_srcreg2;
  rtx ifexpr;
  rtx cond;
  rtx new_set;

  gcc_assert (GET_CODE (set_insn) == SET);
401

402 403 404 405
  cond = XEXP (SET_SRC (set_insn), 0);
  dstreg = SET_DEST (set_insn);
  srcreg = XEXP (SET_SRC (set_insn), 1);
  srcreg2 = XEXP (SET_SRC (set_insn), 2);
406 407 408 409 410
  /* If the conditional move already has the right or wider mode,
     there is nothing to do.  */
  if (GET_MODE_SIZE (GET_MODE (dstreg)) >= GET_MODE_SIZE (cand->mode))
    return true;

411 412 413 414
  map_srcreg = gen_rtx_REG (cand->mode, REGNO (srcreg));
  map_srcreg2 = gen_rtx_REG (cand->mode, REGNO (srcreg2));
  map_dstreg = gen_rtx_REG (cand->mode, REGNO (dstreg));
  ifexpr = gen_rtx_IF_THEN_ELSE (cand->mode, cond, map_srcreg, map_srcreg2);
415 416 417 418 419 420
  new_set = gen_rtx_SET (VOIDmode, map_dstreg, ifexpr);

  if (validate_change (def_insn, &PATTERN (def_insn), new_set, true))
    {
      if (dump_file)
        {
421 422
          fprintf (dump_file,
		   "Mode of conditional move instruction extended:\n");
423 424 425 426
          print_rtl_single (dump_file, def_insn);
        }
      return true;
    }
427 428

  return false;
429 430
}

431 432 433 434
/* Get all the reaching definitions of an instruction.  The definitions are
   desired for REG used in INSN.  Return the definition list or NULL if a
   definition is missing.  If DEST is non-NULL, additionally push the INSN
   of the definitions onto DEST.  */
435

436
static struct df_link *
David Malcolm committed
437
get_defs (rtx_insn *insn, rtx reg, vec<rtx_insn *> *dest)
438
{
439
  df_ref use;
440
  struct df_link *ref_chain, *ref_link;
441

442
  FOR_EACH_INSN_USE (use, insn)
443
    {
444
      if (GET_CODE (DF_REF_REG (use)) == SUBREG)
445
        return NULL;
446 447
      if (REGNO (DF_REF_REG (use)) == REGNO (reg))
	break;
448 449
    }

450
  gcc_assert (use != NULL);
451

452
  ref_chain = DF_REF_CHAIN (use);
453 454

  for (ref_link = ref_chain; ref_link; ref_link = ref_link->next)
455 456
    {
      /* Problem getting some definition for this instruction.  */
457 458 459 460
      if (ref_link->ref == NULL)
        return NULL;
      if (DF_REF_INSN_INFO (ref_link->ref) == NULL)
        return NULL;
461 462
    }

463 464
  if (dest)
    for (ref_link = ref_chain; ref_link; ref_link = ref_link->next)
465
      dest->safe_push (DF_REF_INSN (ref_link->ref));
466

467
  return ref_chain;
468 469
}

470 471 472
/* Return true if INSN is
     (SET (reg REGNO (def_reg)) (if_then_else (cond) (REG x1) (REG x2)))
   and store x1 and x2 in REG_1 and REG_2.  */
473

474
static bool
David Malcolm committed
475
is_cond_copy_insn (rtx_insn *insn, rtx *reg1, rtx *reg2)
476
{
477
  rtx expr = single_set (insn);
478

479 480
  if (expr != NULL_RTX
      && GET_CODE (expr) == SET
481 482 483
      && GET_CODE (SET_DEST (expr)) == REG
      && GET_CODE (SET_SRC (expr))  == IF_THEN_ELSE
      && GET_CODE (XEXP (SET_SRC (expr), 1)) == REG
484
      && GET_CODE (XEXP (SET_SRC (expr), 2)) == REG)
485
    {
486 487 488
      *reg1 = XEXP (SET_SRC (expr), 1);
      *reg2 = XEXP (SET_SRC (expr), 2);
      return true;
489 490
    }

491
  return false;
492 493
}

494 495 496 497 498 499 500 501 502 503
enum ext_modified_kind
{
  /* The insn hasn't been modified by ree pass yet.  */
  EXT_MODIFIED_NONE,
  /* Changed into zero extension.  */
  EXT_MODIFIED_ZEXT,
  /* Changed into sign extension.  */
  EXT_MODIFIED_SEXT
};

504
struct ATTRIBUTE_PACKED ext_modified
505 506 507 508 509 510 511
{
  /* Mode from which ree has zero or sign extended the destination.  */
  ENUM_BITFIELD(machine_mode) mode : 8;

  /* Kind of modification of the insn.  */
  ENUM_BITFIELD(ext_modified_kind) kind : 2;

512 513
  unsigned int do_not_reextend : 1;

514 515 516 517 518 519 520
  /* True if the insn is scheduled to be deleted.  */
  unsigned int deleted : 1;
};

/* Vectors used by combine_reaching_defs and its helpers.  */
typedef struct ext_state
{
521
  /* In order to avoid constant alloc/free, we keep these
522
     4 vectors live through the entire find_and_remove_re and just
523
     truncate them each time.  */
David Malcolm committed
524 525 526 527
  vec<rtx_insn *> defs_list;
  vec<rtx_insn *> copies_list;
  vec<rtx_insn *> modified_list;
  vec<rtx_insn *> work_list;
528 529 530 531 532 533 534

  /* For instructions that have been successfully modified, this is
     the original mode from which the insn is extending and
     kind of extension.  */
  struct ext_modified *modified;
} ext_state;

535 536
/* Reaching Definitions of the extended register could be conditional copies
   or regular definitions.  This function separates the two types into two
537 538 539 540 541 542 543 544 545 546 547
   lists, STATE->DEFS_LIST and STATE->COPIES_LIST.  This is necessary because,
   if a reaching definition is a conditional copy, merging the extension with
   this definition is wrong.  Conditional copies are merged by transitively
   merging their definitions.  The defs_list is populated with all the reaching
   definitions of the extension instruction (EXTEND_INSN) which must be merged
   with an extension.  The copies_list contains all the conditional moves that
   will later be extended into a wider mode conditional move if all the merges
   are successful.  The function returns false upon failure, true upon
   success.  */

static bool
David Malcolm committed
548
make_defs_and_copies_lists (rtx_insn *extend_insn, const_rtx set_pat,
549
			    ext_state *state)
550
{
551
  rtx src_reg = XEXP (SET_SRC (set_pat), 0);
552
  bool *is_insn_visited;
553 554
  bool ret = true;

555
  state->work_list.truncate (0);
556

557
  /* Initialize the work list.  */
558 559
  if (!get_defs (extend_insn, src_reg, &state->work_list))
    gcc_unreachable ();
560

561
  is_insn_visited = XCNEWVEC (bool, max_insn_uid);
562 563

  /* Perform transitive closure for conditional copies.  */
564
  while (!state->work_list.is_empty ())
565
    {
David Malcolm committed
566
      rtx_insn *def_insn = state->work_list.pop ();
567 568
      rtx reg1, reg2;

569 570 571
      gcc_assert (INSN_UID (def_insn) < max_insn_uid);

      if (is_insn_visited[INSN_UID (def_insn)])
572
	continue;
573 574
      is_insn_visited[INSN_UID (def_insn)] = true;

575 576 577
      if (is_cond_copy_insn (def_insn, &reg1, &reg2))
	{
	  /* Push it onto the copy list first.  */
578
	  state->copies_list.safe_push (def_insn);
579 580

	  /* Now perform the transitive closure.  */
581 582
	  if (!get_defs (def_insn, reg1, &state->work_list)
	      || !get_defs (def_insn, reg2, &state->work_list))
583
	    {
584
	      ret = false;
585 586
	      break;
	    }
587 588
        }
      else
589
	state->defs_list.safe_push (def_insn);
590 591 592
    }

  XDELETEVEC (is_insn_visited);
593 594

  return ret;
595 596
}

597 598 599 600 601
/* If DEF_INSN has single SET expression, possibly buried inside
   a PARALLEL, return the address of the SET expression, else
   return NULL.  This is similar to single_set, except that
   single_set allows multiple SETs when all but one is dead.  */
static rtx *
David Malcolm committed
602
get_sub_rtx (rtx_insn *def_insn)
603
{
604 605
  enum rtx_code code = GET_CODE (PATTERN (def_insn));
  rtx *sub_rtx = NULL;
606 607 608

  if (code == PARALLEL)
    {
609
      for (int i = 0; i < XVECLEN (PATTERN (def_insn), 0); i++)
610
        {
611
          rtx s_expr = XVECEXP (PATTERN (def_insn), 0, i);
612 613 614 615 616 617 618 619
          if (GET_CODE (s_expr) != SET)
            continue;

          if (sub_rtx == NULL)
            sub_rtx = &XVECEXP (PATTERN (def_insn), 0, i);
          else
            {
              /* PARALLEL with multiple SETs.  */
620
              return NULL;
621 622 623 624 625 626 627 628
            }
        }
    }
  else if (code == SET)
    sub_rtx = &PATTERN (def_insn);
  else
    {
      /* It is not a PARALLEL or a SET, what could it be ? */
629
      return NULL;
630 631 632
    }

  gcc_assert (sub_rtx != NULL);
633 634 635 636 637 638 639
  return sub_rtx;
}

/* Merge the DEF_INSN with an extension.  Calls combine_set_extension
   on the SET pattern.  */

static bool
David Malcolm committed
640
merge_def_and_ext (ext_cand *cand, rtx_insn *def_insn, ext_state *state)
641 642 643 644 645 646 647 648 649
{
  enum machine_mode ext_src_mode;
  rtx *sub_rtx;

  ext_src_mode = GET_MODE (XEXP (SET_SRC (cand->expr), 0));
  sub_rtx = get_sub_rtx (def_insn);

  if (sub_rtx == NULL)
    return false;
650

651 652 653 654 655 656 657
  if (REG_P (SET_DEST (*sub_rtx))
      && (GET_MODE (SET_DEST (*sub_rtx)) == ext_src_mode
	  || ((state->modified[INSN_UID (def_insn)].kind
	       == (cand->code == ZERO_EXTEND
		   ? EXT_MODIFIED_ZEXT : EXT_MODIFIED_SEXT))
	      && state->modified[INSN_UID (def_insn)].mode
		 == ext_src_mode)))
658
    {
659 660 661 662 663 664 665 666 667 668 669 670 671
      if (GET_MODE_SIZE (GET_MODE (SET_DEST (*sub_rtx)))
	  >= GET_MODE_SIZE (cand->mode))
	return true;
      /* If def_insn is already scheduled to be deleted, don't attempt
	 to modify it.  */
      if (state->modified[INSN_UID (def_insn)].deleted)
	return false;
      if (combine_set_extension (cand, def_insn, sub_rtx))
	{
	  if (state->modified[INSN_UID (def_insn)].kind == EXT_MODIFIED_NONE)
	    state->modified[INSN_UID (def_insn)].mode = ext_src_mode;
	  return true;
	}
672
    }
673 674

  return false;
675 676
}

677 678 679 680 681 682 683 684 685 686 687 688
/* Given SRC, which should be one or more extensions of a REG, strip
   away the extensions and return the REG.  */

static inline rtx
get_extended_src_reg (rtx src)
{
  while (GET_CODE (src) == SIGN_EXTEND || GET_CODE (src) == ZERO_EXTEND)
    src = XEXP (src, 0);
  gcc_assert (REG_P (src));
  return src;
}

689
/* This function goes through all reaching defs of the source
690 691 692 693 694 695 696
   of the candidate for elimination (CAND) and tries to combine
   the extension with the definition instruction.  The changes
   are made as a group so that even if one definition cannot be
   merged, all reaching definitions end up not being merged.
   When a conditional copy is encountered, merging is attempted
   transitively on its definitions.  It returns true upon success
   and false upon failure.  */
697 698

static bool
699
combine_reaching_defs (ext_cand *cand, const_rtx set_pat, ext_state *state)
700
{
David Malcolm committed
701
  rtx_insn *def_insn;
702 703 704
  bool merge_successful = true;
  int i;
  int defs_ix;
705
  bool outcome;
706

707 708
  state->defs_list.truncate (0);
  state->copies_list.truncate (0);
709

710
  outcome = make_defs_and_copies_lists (cand->insn, set_pat, state);
711

712 713
  if (!outcome)
    return false;
714

715 716
  /* If the destination operand of the extension is a different
     register than the source operand, then additional restrictions
717 718
     are needed.  Note we have to handle cases where we have nested
     extensions in the source operand.  */
719 720 721 722
  bool copy_needed
    = (REGNO (SET_DEST (PATTERN (cand->insn)))
       != REGNO (get_extended_src_reg (SET_SRC (PATTERN (cand->insn)))));
  if (copy_needed)
723 724 725 726 727 728
    {
      /* In theory we could handle more than one reaching def, it
	 just makes the code to update the insn stream more complex.  */
      if (state->defs_list.length () != 1)
	return false;

729 730
      /* We require the candidate not already be modified.  It may,
	 for example have been changed from a (sign_extend (reg))
731
	 into (zero_extend (sign_extend (reg))).
732 733 734 735

	 Handling that case shouldn't be terribly difficult, but the code
	 here and the code to emit copies would need auditing.  Until
	 we see a need, this is the safe thing to do.  */
736 737 738
      if (state->modified[INSN_UID (cand->insn)].kind != EXT_MODIFIED_NONE)
	return false;

739 740 741 742 743 744 745 746 747 748 749 750
      /* Transformation of
	 (set (reg1) (expression))
	 (set (reg2) (any_extend (reg1)))
	 into
	 (set (reg2) (any_extend (expression)))
	 (set (reg1) (reg2))
	 is only valid for scalar integral modes, as it relies on the low
	 subreg of reg1 to have the value of (expression), which is not true
	 e.g. for vector modes.  */
      if (!SCALAR_INT_MODE_P (GET_MODE (SET_DEST (PATTERN (cand->insn)))))
	return false;

751 752 753 754 755 756 757 758
      enum machine_mode dst_mode = GET_MODE (SET_DEST (PATTERN (cand->insn)));
      rtx src_reg = get_extended_src_reg (SET_SRC (PATTERN (cand->insn)));

      /* Ensure the number of hard registers of the copy match.  */
      if (HARD_REGNO_NREGS (REGNO (src_reg), dst_mode)
	  != HARD_REGNO_NREGS (REGNO (src_reg), GET_MODE (src_reg)))
	return false;

759
      /* There's only one reaching def.  */
David Malcolm committed
760
      rtx_insn *def_insn = state->defs_list[0];
761 762 763 764 765 766 767

      /* The defining statement must not have been modified either.  */
      if (state->modified[INSN_UID (def_insn)].kind != EXT_MODIFIED_NONE)
	return false;

      /* The defining statement and candidate insn must be in the same block.
	 This is merely to keep the test for safety and updating the insn
768 769 770 771
	 stream simple.  Also ensure that within the block the candidate
	 follows the defining insn.  */
      if (BLOCK_FOR_INSN (cand->insn) != BLOCK_FOR_INSN (def_insn)
	  || DF_INSN_LUID (def_insn) > DF_INSN_LUID (cand->insn))
772 773 774 775 776
	return false;

      /* If there is an overlap between the destination of DEF_INSN and
	 CAND->insn, then this transformation is not safe.  Note we have
	 to test in the widened mode.  */
777 778 779 780 781
      rtx *dest_sub_rtx = get_sub_rtx (def_insn);
      if (dest_sub_rtx == NULL
	  || !REG_P (SET_DEST (*dest_sub_rtx)))
	return false;

782
      rtx tmp_reg = gen_rtx_REG (GET_MODE (SET_DEST (PATTERN (cand->insn))),
783
				 REGNO (SET_DEST (*dest_sub_rtx)));
784 785 786 787 788 789 790 791 792 793
      if (reg_overlap_mentioned_p (tmp_reg, SET_DEST (PATTERN (cand->insn))))
	return false;

      /* The destination register of the extension insn must not be
	 used or set between the def_insn and cand->insn exclusive.  */
      if (reg_used_between_p (SET_DEST (PATTERN (cand->insn)),
			      def_insn, cand->insn)
	  || reg_set_between_p (SET_DEST (PATTERN (cand->insn)),
				def_insn, cand->insn))
	return false;
794 795 796 797 798 799

      /* We must be able to copy between the two registers.   Generate,
	 recognize and verify constraints of the copy.  Also fail if this
	 generated more than one insn.

         This generates garbage since we throw away the insn when we're
Jeff Law committed
800 801 802 803 804
	 done, only to recreate it later if this test was successful. 

	 Make sure to get the mode from the extension (cand->insn).  This
	 is different than in the code to emit the copy as we have not
	 modified the defining insn yet.  */
805 806
      start_sequence ();
      rtx pat = PATTERN (cand->insn);
Jeff Law committed
807
      rtx new_dst = gen_rtx_REG (GET_MODE (SET_DEST (pat)),
808
                                 REGNO (get_extended_src_reg (SET_SRC (pat))));
Jeff Law committed
809
      rtx new_src = gen_rtx_REG (GET_MODE (SET_DEST (pat)),
810 811 812
                                 REGNO (SET_DEST (pat)));
      emit_move_insn (new_dst, new_src);

813
      rtx_insn *insn = get_insns();
814 815 816 817 818 819 820 821
      end_sequence ();
      if (NEXT_INSN (insn))
	return false;
      if (recog_memoized (insn) == -1)
	return false;
      extract_insn (insn);
      if (!constrain_operands (1))
	return false;
822 823 824
    }


825 826 827 828 829 830 831 832 833 834 835 836 837 838 839 840 841 842
  /* If cand->insn has been already modified, update cand->mode to a wider
     mode if possible, or punt.  */
  if (state->modified[INSN_UID (cand->insn)].kind != EXT_MODIFIED_NONE)
    {
      enum machine_mode mode;
      rtx set;

      if (state->modified[INSN_UID (cand->insn)].kind
	  != (cand->code == ZERO_EXTEND
	      ? EXT_MODIFIED_ZEXT : EXT_MODIFIED_SEXT)
	  || state->modified[INSN_UID (cand->insn)].mode != cand->mode
	  || (set = single_set (cand->insn)) == NULL_RTX)
	return false;
      mode = GET_MODE (SET_DEST (set));
      gcc_assert (GET_MODE_SIZE (mode) >= GET_MODE_SIZE (cand->mode));
      cand->mode = mode;
    }

843 844 845 846
  merge_successful = true;

  /* Go through the defs vector and try to merge all the definitions
     in this vector.  */
847 848
  state->modified_list.truncate (0);
  FOR_EACH_VEC_ELT (state->defs_list, defs_ix, def_insn)
849
    {
850
      if (merge_def_and_ext (cand, def_insn, state))
851
	state->modified_list.safe_push (def_insn);
852 853 854 855 856 857 858 859 860 861 862
      else
        {
          merge_successful = false;
          break;
        }
    }

  /* Now go through the conditional copies vector and try to merge all
     the copies in this vector.  */
  if (merge_successful)
    {
863
      FOR_EACH_VEC_ELT (state->copies_list, i, def_insn)
864
        {
865
          if (transform_ifelse (cand, def_insn))
866
	    state->modified_list.safe_push (def_insn);
867 868 869 870 871 872 873 874 875 876
          else
            {
              merge_successful = false;
              break;
            }
        }
    }

  if (merge_successful)
    {
877 878 879 880 881
      /* Commit the changes here if possible
	 FIXME: It's an all-or-nothing scenario.  Even if only one definition
	 cannot be merged, we entirely give up.  In the future, we should allow
	 extensions to be partially eliminated along those paths where the
	 definitions could be merged.  */
882 883 884
      if (apply_change_group ())
        {
          if (dump_file)
885
            fprintf (dump_file, "All merges were successful.\n");
886

887
	  FOR_EACH_VEC_ELT (state->modified_list, i, def_insn)
888 889 890 891 892
	    {
	      ext_modified *modified = &state->modified[INSN_UID (def_insn)];
	      if (modified->kind == EXT_MODIFIED_NONE)
		modified->kind = (cand->code == ZERO_EXTEND ? EXT_MODIFIED_ZEXT
						            : EXT_MODIFIED_SEXT);
893

894 895 896
	      if (copy_needed)
		modified->do_not_reextend = 1;
	    }
897 898 899 900
          return true;
        }
      else
        {
901 902
          /* Changes need not be cancelled explicitly as apply_change_group
             does it.  Print list of definitions in the dump_file for debug
903
             purposes.  This extension cannot be deleted.  */
904 905
          if (dump_file)
            {
906 907
	      fprintf (dump_file,
		       "Merge cancelled, non-mergeable definitions:\n");
908
	      FOR_EACH_VEC_ELT (state->modified_list, i, def_insn)
909
	        print_rtl_single (dump_file, def_insn);
910 911 912 913 914 915 916 917 918 919 920 921
            }
        }
    }
  else
    {
      /* Cancel any changes that have been made so far.  */
      cancel_changes (0);
    }

  return false;
}

922
/* Add an extension pattern that could be eliminated.  */
923 924

static void
David Malcolm committed
925
add_removable_extension (const_rtx expr, rtx_insn *insn,
926
			 vec<ext_cand> *insn_list,
927
			 unsigned *def_map)
928
{
929 930
  enum rtx_code code;
  enum machine_mode mode;
931
  unsigned int idx;
932 933
  rtx src, dest;

934
  /* We are looking for SET (REG N) (ANY_EXTEND (REG N)).  */
935 936 937 938
  if (GET_CODE (expr) != SET)
    return;

  src = SET_SRC (expr);
939
  code = GET_CODE (src);
940
  dest = SET_DEST (expr);
941
  mode = GET_MODE (dest);
942 943

  if (REG_P (dest)
944
      && (code == SIGN_EXTEND || code == ZERO_EXTEND)
945
      && REG_P (XEXP (src, 0)))
946
    {
947 948 949 950
      struct df_link *defs, *def;
      ext_cand *cand;

      /* First, make sure we can get all the reaching definitions.  */
951
      defs = get_defs (insn, XEXP (src, 0), NULL);
952
      if (!defs)
953
	{
954 955 956
	  if (dump_file)
	    {
	      fprintf (dump_file, "Cannot eliminate extension:\n");
957
	      print_rtl_single (dump_file, insn);
958 959 960
	      fprintf (dump_file, " because of missing definition(s)\n");
	    }
	  return;
961
	}
962 963 964 965

      /* Second, make sure the reaching definitions don't feed another and
	 different extension.  FIXME: this obviously can be improved.  */
      for (def = defs; def; def = def->next)
966
	if ((idx = def_map[INSN_UID (DF_REF_INSN (def->ref))])
967
	    && (cand = &(*insn_list)[idx - 1])
968
	    && cand->code != code)
969 970 971 972
	  {
	    if (dump_file)
	      {
	        fprintf (dump_file, "Cannot eliminate extension:\n");
973
		print_rtl_single (dump_file, insn);
974 975 976 977 978 979 980
	        fprintf (dump_file, " because of other extension\n");
	      }
	    return;
	  }

      /* Then add the candidate to the list and insert the reaching definitions
         into the definition map.  */
981
      ext_cand e = {expr, code, mode, insn};
982 983
      insn_list->safe_push (e);
      idx = insn_list->length ();
984 985

      for (def = defs; def; def = def->next)
986
	def_map[INSN_UID (DF_REF_INSN (def->ref))] = idx;
987 988 989
    }
}

990
/* Traverse the instruction stream looking for extensions and return the
991
   list of candidates.  */
992

993
static vec<ext_cand>
994
find_removable_extensions (void)
995
{
996
  vec<ext_cand> insn_list = vNULL;
997
  basic_block bb;
David Malcolm committed
998 999
  rtx_insn *insn;
  rtx set;
1000
  unsigned *def_map = XCNEWVEC (unsigned, max_insn_uid);
1001

1002
  FOR_EACH_BB_FN (bb, cfun)
1003 1004 1005 1006
    FOR_BB_INSNS (bb, insn)
      {
	if (!NONDEBUG_INSN_P (insn))
	  continue;
1007

1008 1009 1010 1011
	set = single_set (insn);
	if (set == NULL_RTX)
	  continue;
	add_removable_extension (set, insn, &insn_list, def_map);
1012 1013
      }

1014
  XDELETEVEC (def_map);
1015

1016
  return insn_list;
1017 1018 1019
}

/* This is the main function that checks the insn stream for redundant
1020
   extensions and tries to remove them if possible.  */
1021

1022
static void
1023
find_and_remove_re (void)
1024
{
1025
  ext_cand *curr_cand;
David Malcolm committed
1026
  rtx_insn *curr_insn = NULL;
1027
  int num_re_opportunities = 0, num_realized = 0, i;
1028
  vec<ext_cand> reinsn_list;
David Malcolm committed
1029 1030
  auto_vec<rtx_insn *> reinsn_del_list;
  auto_vec<rtx_insn *> reinsn_copy_list;
1031
  ext_state state;
1032 1033

  /* Construct DU chain to get all reaching definitions of each
1034
     extension instruction.  */
1035
  df_set_flags (DF_RD_PRUNE_DEAD_DEFS);
1036 1037
  df_chain_add_problem (DF_UD_CHAIN + DF_DU_CHAIN);
  df_analyze ();
1038
  df_set_flags (DF_DEFER_INSN_RESCAN);
1039 1040

  max_insn_uid = get_max_uid ();
1041
  reinsn_list = find_removable_extensions ();
1042 1043 1044 1045 1046
  state.defs_list.create (0);
  state.copies_list.create (0);
  state.modified_list.create (0);
  state.work_list.create (0);
  if (reinsn_list.is_empty ())
1047 1048 1049
    state.modified = NULL;
  else
    state.modified = XCNEWVEC (struct ext_modified, max_insn_uid);
1050

1051
  FOR_EACH_VEC_ELT (reinsn_list, i, curr_cand)
1052
    {
1053
      num_re_opportunities++;
1054

1055
      /* Try to combine the extension with the definition.  */
1056 1057
      if (dump_file)
        {
1058 1059
          fprintf (dump_file, "Trying to eliminate extension:\n");
          print_rtl_single (dump_file, curr_cand->insn);
1060 1061
        }

1062
      if (combine_reaching_defs (curr_cand, curr_cand->expr, &state))
1063 1064
        {
          if (dump_file)
1065
            fprintf (dump_file, "Eliminated the extension.\n");
1066
          num_realized++;
1067 1068 1069 1070 1071 1072 1073
	  /* If the RHS of the current candidate is not (extend (reg)), then
	     we do not allow the optimization of extensions where
	     the source and destination registers do not match.  Thus
	     checking REG_P here is correct.  */
	  if (REG_P (XEXP (SET_SRC (PATTERN (curr_cand->insn)), 0))
	      && (REGNO (SET_DEST (PATTERN (curr_cand->insn)))
		  != REGNO (XEXP (SET_SRC (PATTERN (curr_cand->insn)), 0))))
1074 1075 1076 1077 1078
	    {
              reinsn_copy_list.safe_push (curr_cand->insn);
              reinsn_copy_list.safe_push (state.defs_list[0]);
	    }
	  reinsn_del_list.safe_push (curr_cand->insn);
1079
	  state.modified[INSN_UID (curr_cand->insn)].deleted = 1;
1080 1081 1082
        }
    }

1083 1084 1085 1086 1087 1088 1089 1090 1091 1092 1093 1094 1095 1096 1097 1098
  /* The copy list contains pairs of insns which describe copies we
     need to insert into the INSN stream.

     The first insn in each pair is the extension insn, from which
     we derive the source and destination of the copy.

     The second insn in each pair is the memory reference where the
     extension will ultimately happen.  We emit the new copy
     immediately after this insn.

     It may first appear that the arguments for the copy are reversed.
     Remember that the memory reference will be changed to refer to the
     destination of the extention.  So we're actually emitting a copy
     from the new destination to the old destination.  */
  for (unsigned int i = 0; i < reinsn_copy_list.length (); i += 2)
    {
David Malcolm committed
1099 1100
      rtx_insn *curr_insn = reinsn_copy_list[i];
      rtx_insn *def_insn = reinsn_copy_list[i + 1];
1101 1102 1103 1104 1105 1106

      /* Use the mode of the destination of the defining insn
	 for the mode of the copy.  This is necessary if the
	 defining insn was used to eliminate a second extension
	 that was wider than the first.  */
      rtx sub_rtx = *get_sub_rtx (def_insn);
1107
      rtx pat = PATTERN (curr_insn);
1108
      rtx new_dst = gen_rtx_REG (GET_MODE (SET_DEST (sub_rtx)),
1109
				 REGNO (XEXP (SET_SRC (pat), 0)));
1110 1111 1112 1113
      rtx new_src = gen_rtx_REG (GET_MODE (SET_DEST (sub_rtx)),
				 REGNO (SET_DEST (pat)));
      rtx set = gen_rtx_SET (VOIDmode, new_dst, new_src);
      emit_insn_after (set, def_insn);
1114 1115
    }

1116
  /* Delete all useless extensions here in one sweep.  */
1117
  FOR_EACH_VEC_ELT (reinsn_del_list, i, curr_insn)
1118
    delete_insn (curr_insn);
1119

1120 1121 1122 1123 1124
  reinsn_list.release ();
  state.defs_list.release ();
  state.copies_list.release ();
  state.modified_list.release ();
  state.work_list.release ();
1125
  XDELETEVEC (state.modified);
1126

1127
  if (dump_file && num_re_opportunities > 0)
1128 1129
    fprintf (dump_file, "Elimination opportunities = %d realized = %d\n",
	     num_re_opportunities, num_realized);
1130 1131
}

1132
/* Find and remove redundant extensions.  */
1133 1134

static unsigned int
1135
rest_of_handle_ree (void)
1136
{
1137 1138 1139
  timevar_push (TV_REE);
  find_and_remove_re ();
  timevar_pop (TV_REE);
1140 1141 1142
  return 0;
}

1143 1144 1145
namespace {

const pass_data pass_data_ree =
1146
{
1147 1148 1149 1150 1151 1152 1153 1154
  RTL_PASS, /* type */
  "ree", /* name */
  OPTGROUP_NONE, /* optinfo_flags */
  TV_REE, /* tv_id */
  0, /* properties_required */
  0, /* properties_provided */
  0, /* properties_destroyed */
  0, /* todo_flags_start */
1155
  TODO_df_finish, /* todo_flags_finish */
1156
};
1157 1158 1159 1160

class pass_ree : public rtl_opt_pass
{
public:
1161 1162
  pass_ree (gcc::context *ctxt)
    : rtl_opt_pass (pass_data_ree, ctxt)
1163 1164 1165
  {}

  /* opt_pass methods: */
1166
  virtual bool gate (function *) { return (optimize > 0 && flag_ree); }
1167
  virtual unsigned int execute (function *) { return rest_of_handle_ree (); }
1168 1169 1170 1171 1172 1173 1174 1175 1176 1177

}; // class pass_ree

} // anon namespace

rtl_opt_pass *
make_pass_ree (gcc::context *ctxt)
{
  return new pass_ree (ctxt);
}