fix: give a type to bad cond expr with known condition
commitbe35da05f38df984d518bc4670b36aa0b7ad4185
authorLuc Van Oostenryck <luc.vanoostenryck@gmail.com>
Thu, 3 Aug 2017 23:32:22 +0000 (4 01:32 +0200)
committerLuc Van Oostenryck <luc.vanoostenryck@gmail.com>
Fri, 4 Aug 2017 13:49:49 +0000 (4 15:49 +0200)
tree084b3d919d888f394b51524e24c5454d48cb59ec
parent69e12d779f77120ed7be3b7c67c501e4aa953bd3
fix: give a type to bad cond expr with known condition

Conditional expressions whose second & third operands
have non-compatible types are not conform to the C standard
and sparse emit a warning for them and return the expression
as being erroneous. In consequence, such expressions are not
given a type. This, in turn, makes that some further processing
cannot be done (correctly).

It seems that neither GCC nor clang emit a warning when
there is a type mismatch but the condition is a constant.

In the case we're interested here (the slow compilation of a file)
the operation that cannot be done is the expansion its operands.
This, in turn and among other things, makes that builtins like
__builtin_constant_p() are not evaluated with disatrous consequence
for the amount of work done in the next phases.

Fix this by giving to conditional expressions with constant
condition the same type as the operand selected by the conditional
(but keeping the warning) as GCC & clang seems to do.

Signed-off-by: Luc Van Oostenryck <luc.vanoostenryck@gmail.com>
evaluate.c
validation/cond-err-expand.c [new file with mode: 0644]