Re: [PATCH] if-to-switch pass

2013-07-01 Thread Tom de Vries
On 24/06/13 14:28, Richard Biener wrote: On Fri, Jun 21, 2013 at 10:37 PM, Cesar Philippidis cesar_philippi...@mentor.com wrote: Here is an updated version of Tom's if-to-switch conversion pass that was originally posted here: http://gcc.gnu.org/ml/gcc-patches/2013-01/msg01210.html. I

Re: [PATCH] if-to-switch pass

2013-07-01 Thread Tom de Vries
On 01/07/13 14:02, Tom de Vries wrote: I also dislike the two passes being in early optimizations - that way they do not see the effects of IPA inlining / LTO IPA-CP transforms. I'd rather move it way down towards RTL expansion (though eventually some may say that switch-conversion

Re: [PATCH] if-to-switch pass

2013-06-24 Thread Richard Biener
On Fri, Jun 21, 2013 at 10:37 PM, Cesar Philippidis cesar_philippi...@mentor.com wrote: Here is an updated version of Tom's if-to-switch conversion pass that was originally posted here: http://gcc.gnu.org/ml/gcc-patches/2013-01/msg01210.html. I corrected a build problem with ARM by including

[PATCH] if-to-switch pass

2013-06-21 Thread Cesar Philippidis
Here is an updated version of Tom's if-to-switch conversion pass that was originally posted here: http://gcc.gnu.org/ml/gcc-patches/2013-01/msg01210.html. I corrected a build problem with ARM by including tm_p.h and an ICE caused by NULL_TREE argument being passed to fold_binary () inside