------- Comment #2 from joseph at codesourcery dot com  2008-06-02 19:35 -------
Subject: Re:  Option -mno-isel not working

On Mon, 2 Jun 2008, edmar at freescale dot com wrote:

> The lines deleted in the patch are executed after the command line is parsed.
> The variable rs6000_isel is already properly set at linuspe.h and eabispe.h, 
> so
> no futher action is required.

There are lots of targets using neither of those headers that support E500 
CPUs (every target that uses e500.h), so your patch would stop ISEL being 
enabled for E500 for those targets.

What you want is to set rs6000_isel only if !rs6000_explicit_options.isel.  
(Or find a mask bit to move out of target_flags so ISEL can be enabled 
directly from processor_target_table.)


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=36425

Reply via email to