Re: [PATCH] Fix get_min_precision in __builtin_*_overflow expansion (PR target/66112)

2015-05-12 Thread Richard Biener
On May 12, 2015 8:40:32 PM GMT+02:00, Jakub Jelinek wrote: >Hi! > >This patch fixes a thinko in get_min_precision. If we want UNSIGNED >number and arg_min is signed, but non-negative, calling >wi::min_precision >with SIGNED resulted in one bigger precision than necessary. If >get_min_precision i

[PATCH] Fix get_min_precision in __builtin_*_overflow expansion (PR target/66112)

2015-05-12 Thread Jakub Jelinek
Hi! This patch fixes a thinko in get_min_precision. If we want UNSIGNED number and arg_min is signed, but non-negative, calling wi::min_precision with SIGNED resulted in one bigger precision than necessary. If get_min_precision is originally called with SIGNED and sign is then changed into UNSIG