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