This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: Ping Patch: PR target/24879: SSE3 intrinsic bug
- From: "H. J. Lu" <hjl at lucon dot org>
- To: Ian Lance Taylor <ian at airs dot com>
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Tue, 2 May 2006 11:51:26 -0700
- Subject: Re: Ping Patch: PR target/24879: SSE3 intrinsic bug
- References: <20060116191206.GA12703@lucon.org> <20060217140527.GA17200@lucon.org> <m3odyh3p4j.fsf@gossamer.airs.com>
On Mon, May 01, 2006 at 11:41:16PM -0700, Ian Lance Taylor wrote:
> "H. J. Lu" <hjl@lucon.org> writes:
>
> > On Mon, Jan 16, 2006 at 11:12:06AM -0800, H. J. Lu wrote:
> > > Could someone please comment this:
> > >
> > > http://gcc.gnu.org/ml/gcc-patches/2005-11/msg01195.html
>
> I took a look at this and the follow-on patch
> http://gcc.gnu.org/ml/gcc-patches/2005-11/msg01303.html
>
> I don't understand why the definition of __builtin_ia32_monitor and
> __builtin_ia32_mwait change for 64-bit mode. As I understand it, they
> always take two SImode values. So it seems to me that the type should
> be the same for both, whether you are in 32-bit mode or 64-bit mode.
>
The IA32 SDM isn't very clear for this. I have checked with our chip
people. The 64bit version should be
mwait %rax,%rcx
monitor %eax,%rcx,%rdx
I checked in this patch
http://sourceware.org/ml/binutils/2005-11/msg00302.html
to address it in binutils.
H.J.