Re: [PATCH] bootstrap/49914

2011-08-02 Thread Richard Guenther
On Tue, 2 Aug 2011, Paolo Carlini wrote: > Hi, > > > Ok once abs_hwi materialized. > > Thanks Richard. 'Unfortunately' it materialized already, in the sense that I > wanted to see progress on this issue (also for the sake of a library issue) > and took care of committing the patch adding abs_h

Re: [PATCH] bootstrap/49914

2011-08-02 Thread Paolo Carlini
Hi, > Ok once abs_hwi materialized. Thanks Richard. 'Unfortunately' it materialized already, in the sense that I wanted to see progress on this issue (also for the sake of a library issue) and took care of committing the patch adding abs_hwi as-is, being already approved by Rth, sorry. Do you

Re: [PATCH] bootstrap/49914

2011-08-02 Thread Richard Guenther
On Tue, 2 Aug 2011, Paolo Carlini wrote: > ... oops, due to a missing space I missed an occurrence of abs in > gimple_expand_builtin_pow. I'm restarting bootstrap and test with the below. Ok once abs_hwi materialized. Thanks, Richard. -- Richard Guenther Novell / SUSE Labs SUSE LINUX Products

Re: [PATCH] bootstrap/49914

2011-08-01 Thread Paolo Carlini
... oops, due to a missing space I missed an occurrence of abs in gimple_expand_builtin_pow. I'm restarting bootstrap and test with the below. Paolo. /// Index: fold-const.c === --- fold-const.c(revision 177

[PATCH] bootstrap/49914

2011-08-01 Thread Paolo Carlini
Hi, I bootstrapped and I'm currently finishing testing on x86_64-linux the below. Is it Ok for mainline? Thanks, Paolo. /// 2011-08-02 Paolo Carlini PR bootstrap/49914 * fold-const.c (fold_plusminus_mult_expr): Use abs_hwi instead of abs.