On 11.08.2011, at 13:14, Richard Henderson <r...@twiddle.net> wrote:
> On 08/11/2011 09:38 AM, Blue Swirl wrote: >> On Thu, Aug 11, 2011 at 12:58 PM, Avi Kivity <a...@redhat.com> wrote: >>> On 08/11/2011 03:36 PM, Alexander Graf wrote: >>>> >>>>> >>>>> Or to have automatic generation of the optionals based on the >>>>> primitives, if the optionals are not present. >>>> >>>> That's what's happening in the background already, no? The line is towards >>>> tcg users though, not it tcg internal code. >>>> >>> >>> Yes, and it doesn't make sense to optimize through synthetic instructions. >> >> I'd just create all INDEX_op_* enums and adjust TCG targets etc. to >> call tcg_abort() in the default case. >> > > Seconded. We can still expand the optional enums exactly as we do now, > but make sure that the enum is always present. That'll clean up a *lot* > of ifdefs throughout tcg/*. I agree, but that is a longterm fix. For now, I'd just like to be able to compile again :). So I'd propose to take this patch in for now and then get rid of the op ifdefs later. Alex >