Re: [PATCH 2/4][PR target/88808]Enable bitwise operator for AVX512 masks.

2020-08-20 Thread Hongtao Liu via Gcc-patches
On Wed, Aug 19, 2020 at 2:31 PM Uros Bizjak wrote: > > On Wed, Aug 19, 2020 at 4:17 AM Hongtao Liu wrote: > > OK, modulo: > > +/* { dg-final { scan-assembler-not "%xmm" } } */ > > It is not clear to me what the testcase is testing here. The scan > string is probably too broad and can generate

Re: [PATCH 2/4][PR target/88808]Enable bitwise operator for AVX512 masks.

2020-08-19 Thread Uros Bizjak via Gcc-patches
On Wed, Aug 19, 2020 at 4:17 AM Hongtao Liu wrote: > > On Wed, Aug 19, 2020 at 10:17 AM Hongtao Liu wrote: > > > > On Mon, Aug 17, 2020 at 5:34 PM Uros Bizjak wrote: > > > > > > On Fri, Aug 14, 2020 at 10:24 AM Hongtao Liu wrote: > > > > > > > > Enable direct move between masks and gprs in

Re: [PATCH 2/4][PR target/88808]Enable bitwise operator for AVX512 masks.

2020-08-18 Thread Hongtao Liu via Gcc-patches
On Mon, Aug 17, 2020 at 5:34 PM Uros Bizjak wrote: > > On Fri, Aug 14, 2020 at 10:24 AM Hongtao Liu wrote: > > > > Enable direct move between masks and gprs in pass_reload with > > consideration of cost model. > > > > Changelog > > gcc/ > > * config/i386/i386.c

Re: [PATCH 2/4][PR target/88808]Enable bitwise operator for AVX512 masks.

2020-08-18 Thread Hongtao Liu via Gcc-patches
On Wed, Aug 19, 2020 at 10:17 AM Hongtao Liu wrote: > > On Mon, Aug 17, 2020 at 5:34 PM Uros Bizjak wrote: > > > > On Fri, Aug 14, 2020 at 10:24 AM Hongtao Liu wrote: > > > > > > Enable direct move between masks and gprs in pass_reload with > > > consideration of cost model. > > > > > >

Re: [PATCH 2/4][PR target/88808]Enable bitwise operator for AVX512 masks.

2020-08-17 Thread Uros Bizjak via Gcc-patches
On Fri, Aug 14, 2020 at 10:24 AM Hongtao Liu wrote: > > Enable direct move between masks and gprs in pass_reload with > consideration of cost model. > > Changelog > gcc/ > * config/i386/i386.c (inline_secondary_memory_needed): > No memory is needed between mask regs and gpr. >

[PATCH 2/4][PR target/88808]Enable bitwise operator for AVX512 masks.

2020-08-14 Thread Hongtao Liu via Gcc-patches
Enable direct move between masks and gprs in pass_reload with consideration of cost model. Changelog gcc/ * config/i386/i386.c (inline_secondary_memory_needed): No memory is needed between mask regs and gpr. (ix86_hard_regno_mode_ok): Add condition TARGET_AVX512F for