[Bug gas/12931] ARM: gas fails to set the proper alignment on code sections, causing broken output
https://sourceware.org/bugzilla/show_bug.cgi?id=12931 Andreas Schwab schwab at sourceware dot org changed: What|Removed |Added CC|jackie.rosen at hushmail dot com | -- You are receiving this mail because: You are on the CC list for the bug. ___ bug-binutils mailing list bug-binutils@gnu.org https://lists.gnu.org/mailman/listinfo/bug-binutils
[Bug gas/12931] ARM: gas fails to set the proper alignment on code sections, causing broken output
https://sourceware.org/bugzilla/show_bug.cgi?id=12931 Jackie Rosen jackie.rosen at hushmail dot com changed: What|Removed |Added CC||jackie.rosen at hushmail dot com --- Comment #9 from Jackie Rosen jackie.rosen at hushmail dot com --- *** Bug 260998 has been marked as a duplicate of this bug. *** Seen from the domain http://volichat.com Page where seen: http://volichat.com/adult-chat-rooms Marked for reference. Resolved as fixed @bugzilla. -- You are receiving this mail because: You are on the CC list for the bug. ___ bug-binutils mailing list bug-binutils@gnu.org https://lists.gnu.org/mailman/listinfo/bug-binutils
[Bug gas/12931] ARM: gas fails to set the proper alignment on code sections, causing broken output
http://sourceware.org/bugzilla/show_bug.cgi?id=12931 --- Comment #7 from cvs-commit at gcc dot gnu.org cvs-commit at gcc dot gnu.org 2011-06-30 13:07:24 UTC --- CVSROOT:/cvs/src Module name:src Changes by:ni...@sourceware.org2011-06-30 13:07:21 Modified files: gas/testsuite : ChangeLog gas/testsuite/gas/arm: blx-bad.d inst-po-be.d inst-po.d Log message: PR gas/12931 * gas/arm/blx-bad.d: Add exrta nop at end of disassembly. * gas/arm/inst-po-be.d: Add exrta nop at end of disassembly. * gas/arm/inst-po.d: Add exrta nop at end of disassembly. Patches: http://sourceware.org/cgi-bin/cvsweb.cgi/src/gas/testsuite/ChangeLog.diff?cvsroot=srcr1=1.1925r2=1.1926 http://sourceware.org/cgi-bin/cvsweb.cgi/src/gas/testsuite/gas/arm/blx-bad.d.diff?cvsroot=srcr1=1.2r2=1.3 http://sourceware.org/cgi-bin/cvsweb.cgi/src/gas/testsuite/gas/arm/inst-po-be.d.diff?cvsroot=srcr1=1.1r2=1.2 http://sourceware.org/cgi-bin/cvsweb.cgi/src/gas/testsuite/gas/arm/inst-po.d.diff?cvsroot=srcr1=1.1r2=1.2 -- Configure bugmail: http://sourceware.org/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ bug-binutils mailing list bug-binutils@gnu.org https://lists.gnu.org/mailman/listinfo/bug-binutils
[Bug gas/12931] ARM: gas fails to set the proper alignment on code sections, causing broken output
http://sourceware.org/bugzilla/show_bug.cgi?id=12931 Hans-Peter Nilsson hp at sourceware dot org changed: What|Removed |Added CC||hp at sourceware dot org --- Comment #8 from Hans-Peter Nilsson hp at sourceware dot org 2011-07-01 04:19:31 UTC --- Nick (being already CC:ed on the referred PR) or anyone else ARM-knowledgeable, pretty please have a quick look to see if the fix for this bug was the proper fix for PR sim/12737. Thanks. -- Configure bugmail: http://sourceware.org/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ bug-binutils mailing list bug-binutils@gnu.org https://lists.gnu.org/mailman/listinfo/bug-binutils
[Bug gas/12931] ARM: gas fails to set the proper alignment on code sections, causing broken output
http://sourceware.org/bugzilla/show_bug.cgi?id=12931 --- Comment #5 from Dave Martin dave.martin at linaro dot org 2011-06-29 16:47:08 UTC --- On Wed, Jun 29, 2011 at 5:33 PM, nickc at redhat dot com sourceware-bugzi...@sourceware.org wrote: http://sourceware.org/bugzilla/show_bug.cgi?id=12931 Nick Clifton nickc at redhat dot com changed: What |Removed |Added Status|NEW |RESOLVED CC| |nickc at redhat dot com Resolution| |FIXED --- Comment #4 from Nick Clifton nickc at redhat dot com 2011-06-29 16:33:44 UTC --- Hi Dave, Thanks for reporting this problem and providing a patch to fix it too. I think that 2 byte alignment should be fine for thumb-code-containing sections, but if someone does come up with a scenario where this assumption does not work then we can always fix the assembler again. That seems reasonable -- I think the proposed fix should be a useful interim step, since it's pretty simple and makes the situation no worse. Did the patch look sensible to you? I'm no gas hacker... Cheers ---Dave -- Configure bugmail: http://sourceware.org/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ bug-binutils mailing list bug-binutils@gnu.org https://lists.gnu.org/mailman/listinfo/bug-binutils
[Bug gas/12931] ARM: gas fails to set the proper alignment on code sections, causing broken output
http://sourceware.org/bugzilla/show_bug.cgi?id=12931 --- Comment #3 from cvs-commit at gcc dot gnu.org cvs-commit at gcc dot gnu.org 2011-06-29 16:29:43 UTC --- CVSROOT:/cvs/src Module name:src Changes by:ni...@sourceware.org2011-06-29 16:29:38 Modified files: gas: ChangeLog gas/config : tc-arm.c Log message: PR gas/12931 * config/tc-arm.c (mapping_state): When changing to ARM or THUMB state set the minimum required alignment of the section. Patches: http://sourceware.org/cgi-bin/cvsweb.cgi/src/gas/ChangeLog.diff?cvsroot=srcr1=1.4527r2=1.4528 http://sourceware.org/cgi-bin/cvsweb.cgi/src/gas/config/tc-arm.c.diff?cvsroot=srcr1=1.495r2=1.496 -- Configure bugmail: http://sourceware.org/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ bug-binutils mailing list bug-binutils@gnu.org https://lists.gnu.org/mailman/listinfo/bug-binutils
[Bug gas/12931] ARM: gas fails to set the proper alignment on code sections, causing broken output
http://sourceware.org/bugzilla/show_bug.cgi?id=12931 --- Comment #2 from Dave Martin dave.martin at linaro dot org 2011-06-28 12:57:37 UTC --- Created attachment 5824 -- http://sourceware.org/bugzilla/attachment.cgi?id=5824 possible fix The attached patch hooks into mapping_state() to mark the alignment requirement on the current output section whenever we start emitting instructions. It might be safer to make the alignment 4 for Thumb code as well as ARM code; the current patch gives Thumb code an alignment of 2 bytes, based on the principles that (a) this makes the current situation no worse, and (b) all realistic situations where a Thumb code section requires 4-byte alignment will involve an explicit .align or literal pool somewhere, so we don't need to worry. -- Configure bugmail: http://sourceware.org/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ bug-binutils mailing list bug-binutils@gnu.org https://lists.gnu.org/mailman/listinfo/bug-binutils