]> git.ipfire.org Git - thirdparty/gcc.git/commit - gcc/config/arm/arm.h
re PR target/70496 (inadvertent change to ASM_APP_OFF for .arm / .thumb directives.)
authorRamana Radhakrishnan <ramana.radhakrishnan@arm.com>
Fri, 1 Apr 2016 14:58:53 +0000 (14:58 +0000)
committerRamana Radhakrishnan <ramana@gcc.gnu.org>
Fri, 1 Apr 2016 14:58:53 +0000 (14:58 +0000)
commit41d146597caea6d5cab6def07a8b127de7b2099c
tree5d55f640e781322ec4066f43f995a1399b6a9854
parent9553dd73e12101952610efe89be599e06d999aee
re PR target/70496 (inadvertent change to ASM_APP_OFF for .arm / .thumb directives.)

Fix PR target/70496

While doing the unified asm rewrite - I inadvertently changed the
meaning of ASM_APP_OFF which causes failures when folks who know what
they are doing switch between arm and thumb states within a
function. The intent of the unified asm rewrite was not to affect any
inline assembler code in that it would remain in divided syntax by
default and switching back to unified asm in normal compiled code
after the inline assembler block. Thanks to Jim Wilson for pointing it
out on the linaro list.

Fixed thusly and a test is added. Tested arm-none-eabi cross and
applied.

Ramana

2016-04-01  Ramana Radhakrishnan  <ramana.radhakrishnan@arm.com>

PR target/70496
* config/arm/arm.h (ASM_APP_OFF): Handle TARGET_ARM
and TARGET_THUMB.

2016-04-01  Ramana Radhakrishnan  <ramana.radhakrishnan@arm.com>

PR target/70496
* gcc.target/arm/pr70496.c: New test.

From-SVN: r234675
gcc/ChangeLog
gcc/config/arm/arm.h
gcc/testsuite/ChangeLog
gcc/testsuite/gcc.target/arm/pr70496.c [new file with mode: 0644]