Re: [PATCH i386] Allow cltd/cqto etc on modern CPUs

2012-12-01 Thread Xinliang David Li
Fixed. thanks, David On Sat, Dec 1, 2012 at 4:08 PM, Steven Bosscher wrote: > On Sat, Dec 1, 2012 at 6:50 AM, Xinliang David Li wrote: >> 2010-11-30 Xinliang David Li <> >> >> * config/i386/i386.c: Allow sign extend instructions (cltd etc) >> on modern CPUs. > > You installed

Re: [PATCH i386] Allow cltd/cqto etc on modern CPUs

2012-12-01 Thread Steven Bosscher
On Sat, Dec 1, 2012 at 6:50 AM, Xinliang David Li wrote: > 2010-11-30 Xinliang David Li <> > > * config/i386/i386.c: Allow sign extend instructions (cltd etc) > on modern CPUs. You installed the patch without the ChangeLog entry... (http://gcc.gnu.org/ml/gcc-cvs/2012-12/msg00027.

Re: [PATCH i386] Allow cltd/cqto etc on modern CPUs

2012-12-01 Thread Uros Bizjak
Hello! > The following proposed patch fixed the problem. Note that for Atom, > only the CWD instruction is slow with 5 cycle latency, the rest sign > extension instructions are fast -- the fix for Atom needs finer grain > control and can be done separately. > > 2010-11-30 Xinliang David Li > >

[PATCH i386] Allow cltd/cqto etc on modern CPUs

2012-11-30 Thread Xinliang David Li
Compiling the following code with O2 typedef unsigned long ulong; typedef __SIZE_TYPE__ size_t; long woo_i(long a, long b) { return a/b; } GCC generates: .LFB0: .cfi_startproc movq%rdi, %rdx movq%rdi, %rax sarq$63, %rdx idivq %rsi ret