This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH 2/3] Allow divmod on UNITS_PER_WORD vectors


On Thu, Jul 19, 2012 at 11:52:57AM -0700, Richard Henderson wrote:
> IA64 vectors modes are of course all UNITS_PER_WORD.  I see no reason
> to prevent the transformation of a V4HImode divmod, or even of V2SImode.
> 
> Jakub, was there another reason you added this condition?

I guess I wanted to play on the safe side given that vectorizable_operation
already does something with such mode sizes:

  if (icode == CODE_FOR_nothing)
    {
      if (vect_print_dump_info (REPORT_DETAILS))
        fprintf (vect_dump, "op not supported by target.");
      /* Check only during analysis.  */
      if (GET_MODE_SIZE (vec_mode) != UNITS_PER_WORD
          || (!vec_stmt && vf < vect_min_worthwhile_factor (code)))
        return false;
      if (vect_print_dump_info (REPORT_DETAILS))
        fprintf (vect_dump, "proceeding using word mode.");
    }

and didn't want to convert something that possibly could be already
vectorized somehow, at least without being able to test what it actually
does.

	Jakub


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]