Richard Sandiford <rdsandif...@googlemail.com> writes:
> Robert Suchanek <robert.sucha...@imgtec.com> writes:
> > The patch below disables generation of the branch likely instructions for 
> > -Os
> > but only for generic architecture.  The branch likely may result in some
> > code size reduction but the cost of running the code on R6 core is 
> > significant.
> 
> How about instead splitting PTF_AVOID_BRANCHLIKELY into
> PTF_AVOID_BRANCHLIKELY_SPEED and PTF_AVOID_BRANCHLIKELY_SIZE?
> We could have PTF_AVOID_BRANCHLIKELY_ALWAYS as an OR of the two.

This sounds OK and is nicer.

> Anything that does string ops on the architecture is suspicious :-)

You can blame me for this.  I advocated the string comparison approach as I had 
to
do the same thing in gas IIRC for some feature and couldn't think of anything
better to suggest.

Thanks,
Matthew

Reply via email to