[PATCH, PR43920, 1/9] ARM specific part.

Tom de Vries vries@codesourcery.com
Sat Apr 2 17:06:00 GMT 2011


On 04/02/2011 09:47 AM, Richard Guenther wrote:
> On Fri, Apr 1, 2011 at 6:06 PM, Tom de Vries <vries@codesourcery.com> wrote:
>> On 04/01/2011 05:18 PM, Richard Earnshaw wrote:
>>>
>>> On Fri, 2011-04-01 at 16:45 +0200, Tom de Vries wrote:
>>>> Reposting, with ChangeLog.
>>>
>>>  #define BRANCH_COST(speed_p, predictable_p) \
>>> -  (TARGET_32BIT ? 4 : (optimize > 0 ? 2 : 0))
>>> +  (TARGET_32BIT ? (TARGET_THUMB2 && optimize_size ? 1 : 4) \
>>> +               : (optimize > 0 ? 2 : 0))
>>>
>>> Don't use optimize_size here, use !speed_p.
>>>
>>> Otherwise OK.
>>>
>>
>> Replaced optimize_size by !speed_p.
> 
> I wonder if we can add a code-size test harness.  Using GNU size
> for examle, if available and a new dg-final { object-size SIZE } that
> fails when the size is greater than the specified one (of course all
> object-size tests with specific target restrictions).

like this?

Thanks,
- Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 8_crossjump-fallthru-ml.test.patch
Type: text/x-patch
Size: 3661 bytes
Desc: not available
URL: <http://gcc.gnu.org/pipermail/gcc-patches/attachments/20110402/c0a4551f/attachment.bin>


More information about the Gcc-patches mailing list