Re: [PATCH] stack adjustment is unsigned

2017-05-01 Thread Jeff Law
On 05/01/2017 04:23 PM, Nathan Sidwell wrote: I happened to notice that stack adjustment code was confusingly testing for an unsigned value being > 0. In this case unadjusted_alignment is of type unsigned HOST_WIDE_INT, so the 'else' portion of the has no effect. This patch simplifies the adj

[PATCH] stack adjustment is unsigned

2017-05-01 Thread Nathan Sidwell
I happened to notice that stack adjustment code was confusingly testing for an unsigned value being > 0. In this case unadjusted_alignment is of type unsigned HOST_WIDE_INT, so the 'else' portion of the has no effect. This patch simplifies the adjustment to be less confusing (IMHO). booted an