Re: [PATCH] Improve rotate fold-const pattern matching (PR target/82498)

2017-10-13 Thread Richard Biener
On Thu, 12 Oct 2017, Jakub Jelinek wrote: > Hi! > > Marc in the PR mentioned that it is not really good that the recommended > rotate pattern is recognized only during forwprop1 and later, which is after > einline and that inlining or early opts could have changed stuff too much so > that we

Re: [x86] GFNI enabling[1/4]

2017-10-13 Thread Jakub Jelinek
On Fri, Oct 13, 2017 at 07:03:14AM +, Koval, Julia wrote: --- a/gcc/config/i386/i386.opt +++ b/gcc/config/i386/i386.opt @@ -753,6 +753,10 @@ mrdpid Target Report Mask(ISA_RDPID) Var(ix86_isa_flags2) Save Support RDPID built-in functions and code generation. +mgfni +Target Report

[x86] GFNI enabling[1/4]

2017-10-13 Thread Koval, Julia
Hi, gcc/ * gcc/common/config/i386/i386-common.c (OPTION_MASK_ISA_GFNI_SET, (OPTION_MASK_ISA_GFNI_UNSET): New. (ix86_handle_option): Handle OPT_mgfni. * gcc/config/i386/cpuid.h (bit_GFNI): New. * gcc/config/i386/driver-i386.c (host_detect_local_cpu): Detect

Re: 0004-Part-4.-Update-x86-backend-to-enable-Intel-CET

2017-10-13 Thread Uros Bizjak
On Thu, Oct 12, 2017 at 8:45 PM, Tsimbalist, Igor V wrote: > Uros, > > Attached is an updated patch. The main difference is in option name and > attribute name change. Other code is the same. Trivial changes (if they fall under "trivial" rule) don't need extra

Re: [PATCH] Avoid UB in ia32intrin.h rotate patterns (PR target/82498)

2017-10-13 Thread Uros Bizjak
On Thu, Oct 12, 2017 at 9:39 PM, Jakub Jelinek wrote: > Hi! > > The ia32intrin.h rotate intrinsics require the second argument to be > in between 1 and 31 (or 63), otherwise they invoke UB. But, we can do much > better while generating the same instruction when optimizing, so

Re: [PATCH] Fix various arithmetic patterns with %[abcd]h destination (PR target/82524)

2017-10-13 Thread Uros Bizjak
On Thu, Oct 12, 2017 at 9:49 PM, Jakub Jelinek wrote: > Hi! > > As mentioned in the PR, there are two bugs in these. One is that > the zero_extract destination is effectively another input operand (for the > remaining bits that are unchanged) and thus the constraint can't be

Re: [PATCH] Improve x86 and + rotate (PR target/82498)

2017-10-13 Thread Uros Bizjak
On Thu, Oct 12, 2017 at 9:11 PM, Jakub Jelinek wrote: > On Thu, Oct 12, 2017 at 10:40:22AM +0200, Uros Bizjak wrote: >> > So, if you aren't against it, I can extend the patch to handle the 4 >> > other mask patterns; as for other modes, SImode is what is being handled >> >

<    1   2