On Thu, Sep 6, 2012 at 3:39 PM, Maciej W. Rozycki <ma...@codesourcery.com> wrote: > On Thu, 6 Sep 2012, Andrew Pinski wrote: > >> Could you explain why you are changing system.h ? >> Also seems like TARGET_VLE_ISEL should not be needed TARGET_ISEL is >> always set for VLE targets. > > You mean this: > > + POWERPC_E200_MASK = MASK_VLE | MASK_ISEL | MASK_MULTIPLE > > ? Well, this just marks that the e200 processor supports ISEL regardless > of the mode selected (standard vs VLE). Then with -mvle ISEL is supposed > to be enabled regardless of the processor setting in effect (ISEL is a > part of the base VLE instruction set, while it is optional in the standard > mode).
What I mean is set TARGET_ISEL to true when -mvle is supplied. Thanks, Andrew