This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: PATCH: Add x86 integer intrinsics

From: Paul Brook <paul at codesourcery dot com>
To: gcc-patches at gcc dot gnu dot org
Cc: "H.J. Lu" <hjl dot tools at gmail dot com>, Paolo Bonzini <paolo dot bonzini at gmail dot com>, ubizjak at gmail dot com
Date: Wed, 10 Jun 2009 16:43:52 +0100
Subject: Re: PATCH: Add x86 integer intrinsics
References: <20090610135011.GA14489@lucon.org> <4A2FC862.6070700@gmail.com> <6dc9ffc80906100818sfe7ee4dpbb9416036c437884@mail.gmail.com>

> Well, it isn't the case. At -O2, I got
>
> #include <x86intrin.h>
>
> int
> rol(int x, int y)
> {
>  return (x << y) | (x >> (32 - y));
>  }

Your code is bogus. Use unsigned int.

Paul

References:
- PATCH: Add x86 integer intrinsics
  - From: H.J. Lu
- Re: PATCH: Add x86 integer intrinsics
  - From: Paolo Bonzini
- Re: PATCH: Add x86 integer intrinsics
  - From: H.J. Lu

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]