[PATCH] ARM - gas: Maximum alignment size augmented to 64

[Daniel Gutson]

Thanks Matthias,
   with the latest cvs up, I could reproduce the behavior.

This version of the patch has the dump updated to the new format.

   Daniel.

Matthias Klose wrote:
> The output of objdump did change,
> 
> Disassembly of section .text:
> 00000000 <foo> f04f 0001        mov.w   r0, #1
> 00000004 <foo+0x4> 46c0         nop                     ; (mov r8, r8)
> 
> so the regexps of the form
> 
> 00000000 <foo> f04f 0001.*mov.w.*r0, #1.*; 0x1
> 
> don't match anymore. change this to
> 
> 00000000 <foo> f04f 0001.*mov.w.*r0, #1.*
> 
> for all four occurrances and the test succeeds.
> 
>   Matthias
> 
> Daniel Gutson schrieb:
>> Hi Matthias,
>>
>>   I couldn't get the fail, even after updating my sources from mainline.
>>
>> Could you please provide more details?
>>
>> Thanks,
>>
>>     Daniel.
>>
>>
>> Matthias Klose wrote:
>>> Daniel Gutson schrieb:
>>>> Hi,
>>>>  in the attached patch, the maximum allowed code alignment has been
>>>> increased from
>>>> 32 to 64 bytes. This change affects the .p2align and .align directives
>>>> in GAS and the
>>>> -falign-functions GCC option.
>>>>
>>>> I tested the code by running the gas testsuite, including the new test
>>>> case I added.
>>> the test case does succeed with binutils 20090622 and the patch
>>> applied, it
>>> fails with binutils 20090704 and the patch applied.
>>>
>>>   Matthias
> 

-- 
Daniel Gutson
CodeSourcery
www.codesourcery.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: align64_2.patch
Type: text/x-diff
Size: 9425 bytes
Desc: not available
URL: <https://sourceware.org/pipermail/binutils/attachments/20090713/a3cc3dec/attachment.bin>