Re: [PATCH][RFC][x86] Fix PR91154, add SImode smax, allow SImode add in SSE regs

[Richard Biener <rguenther at suse dot de>]

On Mon, 5 Aug 2019, Uros Bizjak wrote:

> On Mon, Aug 5, 2019 at 2:54 PM Jakub Jelinek <jakub@redhat.com> wrote:
> >
> > On Mon, Aug 05, 2019 at 02:51:01PM +0200, Uros Bizjak wrote:
> > > > (define_mode_iterator MAXMIN_IMODE [SI "TARGET_SSE4_1"] [DI "TARGET_AVX512F"])
> > > >
> > > > and then we need to split DImode for 32bits, too.
> > >
> > > For now, please add "TARGET_64BIT && TARGET_AVX512F" for DImode
> > > condition, I'll provide _doubleword splitter later.
> >
> > Shouldn't that be TARGET_AVX512VL instead?  Or does the insn use %g0 etc.
> > to force use of %zmmN?
> 
> It generates V4SI mode, so - yes, AVX512VL.

    case SMAX:
    case SMIN:
    case UMAX:
    case UMIN:
      if ((mode == DImode && (!TARGET_64BIT || !TARGET_AVX512VL))
          || (mode == SImode && !TARGET_SSE4_1))
        return false;

so there's no way to use AVX512VL for 32bit?

Richard.