This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [trans-mem] rms-tm bug report
- From: Aldy Hernandez <aldyh at redhat dot com>
- To: Patrick Marlier <patrick dot marlier at unine dot ch>
- Cc: Richard Henderson <rth at redhat dot com>, FELBER Pascal <pascal dot felber at unine dot ch>, Javier Arias <javier dot arias at bsc dot es>, "gokcen dot kestor at bsc dot es" <gokcen dot kestor at bsc dot es>, "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
- Date: Tue, 1 Jun 2010 16:04:54 -0400
- Subject: Re: [trans-mem] rms-tm bug report
- References: <4BF4DD38.3080708@unine.ch> <20100524171655.GA19900@redhat.com> <20100524171812.GB19900@redhat.com> <4BFAEE43.40709@redhat.com> <20100525141103.GA26234@redhat.com> <4BFBE254.5010309@redhat.com> <20100528215344.GB21714@redhat.com> <4C004D0F.8040207@redhat.com> <20100531160521.GA3061@redhat.com> <4C04BFE9.60409@unine.ch>
> I had "error: enum conversion in initialization is invalid in C++".
>
> I corrected this by adding a explicit cast (enum ix86_builtins). I
> don't know if BUILT_IN_TM_STORE_M64 must be define in the enum
> ix86_builtins definition.
>
> example:
> { OPTION_MASK_ISA_MMX, 0, "__builtin__ITM_WM64", (enum
> ix86_builtins)BUILT_IN_TM_STORE_M64, UNKNOWN, VOID_FTYPE_PV2SI_V2SI
> },
>
>
> Another error was "error: ISO C90 forbids mixed declarations and code"
Whoops. This happens when you don't bootstrap, because the TM branch
hadn't bootstrapped for the longest time. But now that I'm making non
trans-mem.c changes, so I really should bootstrap.
Thanks for catching this.