On 01/02/2014 05:26 PM, Eric Botcazou wrote:
So, I'd like to ping the original patch and Kenny's patch to resolve the
issues you found. If you have any other concerns or thoughts, let us
know.
Almost OK, but remove the strange quotes in the comment for the INTEGER_CST
case of expand_expr_real_1 (and optionally add the fast path for the common
case written down by Richard).
eric,
the fast path is not any faster. that was why we did not do it. And by
the time you add the tests it would be slower. The thing is that on
the inside of the so called fast path, you are still converting the tree
to wide-int first and that conversion still is checking the rest of
these things.
the quotes will be removed before committing.
kenny
