[arm] PR target/85026: Fix ldrsh length estimate in Thumb state
authorKyrylo Tkachov <kyrylo.tkachov@arm.com>
Fri, 23 Mar 2018 16:43:43 +0000 (16:43 +0000)
committerKyrylo Tkachov <ktkachov@gcc.gnu.org>
Fri, 23 Mar 2018 16:43:43 +0000 (16:43 +0000)
commit63028caaeb63f5a3fa1dcabaae10fe5902a39048
treea60f13219c7d8bb048d9338aa4dbb01fb98eb1e3
parent2fce761ff8de61f2418c36dbba20b3ac63890996
[arm] PR target/85026: Fix ldrsh length estimate in Thumb state

This bug has been reported against GCC 7.3.0 but it is latent in all release branches and on trunk.
We underestimate the length of the LRSH instruction in Thumb state.
Unlike other load instructions LDRSH can be encoded in 16 bits only when using a register offset.
In the testcase we have "ldrsh   r2, [r4]" being assigned a length of 2, which is wrong.
So we don't calculate branch ranges properly and cause the assembler error.

The fix is to make the unaligned_loadhis insn similar to the *arm_extendqihi_insn insn that outputs an LDRSB.
Just remove the wrong 2-byte alternative. I don't think this is worth inventing a new "register-offset-only" constraint.
This also makes the patch safer for backporting.

Bootstrapped and tested on arm-none-linux-gnueabihf.

PR target/85026
* config/arm/arm.md (unaligned_loadhis): Remove first alternative.
Clean up attributes.

* g++.dg/pr85026.C: New test.

From-SVN: r258818
gcc/ChangeLog
gcc/config/arm/arm.md
gcc/testsuite/ChangeLog
gcc/testsuite/g++.dg/pr85026.C [new file with mode: 0644]