Re: [ARM] fix arm bootstrap

[Kyrill Tkachov]

On 15/05/15 07:46, Kugan wrote:

Sorry, this is not the right fix. Looking at it.

Hi Kugan,

FWIW, I'm seeing this bootstrap failure as well.
If you have a patch for this, it would be appreciated.

Kyrill

Thanks,
Kugan

On 15/05/15 16:41, Kugan wrote:

Hi Richard,

r223090 seem to miss a parenthesis and seem to be causing:

                  from ../../widen/gcc/fold-const.c:46:
../../widen/gcc/fold-const.c: In function 'tree_node*
fold_range_test(location_t, tree_code, tree, tree, tree)':
../../widen/gcc/config/arm/arm.h:2068:4: error: enumeral and
non-enumeral type in conditional expression [-Werror=extra]
     ? (TARGET_THUMB ? false : true)     \
     ^
../../widen/gcc/fold-const.c:5234:12: note: in expansion of macro
'LOGICAL_OP_NON_SHORT_CIRCUIT'
    else if (LOGICAL_OP_NON_SHORT_CIRCUIT
             ^
../../widen/gcc/fold-const.c: In function 'tree_node*
fold_truth_andor(location_t, tree_code, tree, tree, tree, tree, tree)':
../../widen/gcc/config/arm/arm.h:2068:4: error: enumeral and
non-enumeral type in conditional expression [-Werror=extra]
     ? (TARGET_THUMB ? false : true)     \
     ^
../../widen/gcc/fold-const.c:8431:7: note: in expansion of macro
'LOGICAL_OP_NON_SHORT_CIRCUIT'
    if (LOGICAL_OP_NON_SHORT_CIRCUIT


Regression testing for arm-none-linux-gnueab is ongoing.
Is this OK for trunk if no regression?

Thanks,
Kugan

gcc/ChangeLog:

2015-05-15  Kugan Vivekanandarajah  <kug...@linaro.org>

        * config/arm/arm.h (enum arm_auto_incmodes): Add parenthesis.