[ARM] Thumb2 replicated constants

Submitted by Andrew Stubbs on Aug. 30, 2011, 3:04 p.m.


Message ID 4E5CFC09.20906@codesourcery.com
State New
Headers show

Commit Message

Andrew Stubbs Aug. 30, 2011, 3:04 p.m.
On 30/08/11 15:32, Andrew Stubbs wrote:
> On 26/08/11 11:03, Andrew Stubbs wrote:
>> There was a bug I found in final testing, so this has been delayed
>> somewhat.
>> I've just committed this version. There are a few minor changes to the
>> way negative/inverted constants are generated.
> Bernd found another bug whist testing for arm. Apparently there was a
> bug that didn't show up in my thumb tests, although I have no clue why?
> Anyway, fixed thusly and committed as obvious (and urgent).

And also there was an issue build with -Werror ... I'm not having much 
luck at the moment. :(

Anyway, likewise fixed and committed as obvious.


Patch hide | download patch | download mbox

2011-08-30  Andrew Stubbs  <ams@codesourcery.com>

	* config/arm/arm.c (optimal_immediate_sequence_1): Make b1, b2,
	b3 and b4 unsigned.

--- a/gcc/config/arm/arm.c
+++ b/gcc/config/arm/arm.c
@@ -2624,7 +2624,7 @@  optimal_immediate_sequence_1 (enum rtx_code code, unsigned HOST_WIDE_INT val,
       int end;
-      int b1, b2, b3, b4;
+      unsigned int b1, b2, b3, b4;
       unsigned HOST_WIDE_INT result;
       int loc;