Patchwork [ARM] Thumb2 replicated constants

login
register
mail settings
Submitter Andrew Stubbs
Date Aug. 30, 2011, 3:04 p.m.
Message ID <4E5CFC09.20906@codesourcery.com>
Download mbox | patch
Permalink /patch/112351/
State New
Headers show

Comments

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.

Andrew

Patch

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

	gcc/
	* 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,
   do
     {
       int end;
-      int b1, b2, b3, b4;
+      unsigned int b1, b2, b3, b4;
       unsigned HOST_WIDE_INT result;
       int loc;