On Mon, Dec 28, 2015 at 04:17:02PM +0300, Kirill Yukhin wrote:
> Hello,
> On 02 Dec 20:00, Kirill Yukhin wrote:
> > Hello,
> > On 30 Nov 13:46, Kirill Yukhin wrote:
> > > Hello,
> > > Patch in the bottom splits masked version of vec_extract_hi_<mode>
> > > to block AVX-1512VL insn generation for KNL and cures ICE on
> > > spec2k6/450.soplex.
> > >
> > > Bootstrapped and regtesed.
> > >
> > > If no objections - I'll commit on Wednesday.
> > >
> > > gcc/
> > > * config/i386/sse.md (define_insn "vec_extract_hi_<mode>_maskm"):
> > > Remove "prefix_extra".
> > > (define_insn "vec_extract_hi_<mode>_mask"): New.
> > > (define_insn "vec_extract_hi_<mode>"): Remove masking.
> > > gcc/testsuite/
> > > * gcc.target/i386/avx512vl-vextractf32x4-1.c: Fix scan pattern.
> Similar patch is needed to make spec2k6/465.tonto working for gcc-5.
> Is patch in the bottom ok for gcc-5-branch if bootstrapped and regtested?
> It cures spec2k6/465.tonto illegal insn emit.
Can you add a runtime testcase that would fail without the patch and succeed
with it? Perhaps add some asms etc. to force the operands to look similarly
for RA purposes. The patch is ok with or without that testcase, though the
testcase would be greatly appreciated.
Jakub
