Re: Suspicious expand_complex_division() in tree-comlpex.c

2007-01-04 Thread Andrew Pinski
> > > > Yes I see now... a quite complicated way to express the choice logic: > > 1. if -fcx-limited-range is set go straight for the quick overflowing > version. > 2. be strict in case of ISO C99. > 3. handle floaing poing divisions more precisely then multiplications > else. if you look

Re: Suspicious expand_complex_division() in tree-comlpex.c

2007-01-04 Thread Marcin Dalecki
Wiadomość napisana w dniu 2007-01-05, o godz08:36, przez Andrew Pinski: Thus my question to whoever looked at this code close enough is: 1. Is the FALLTHRU really OK? yes, see above. Plus the fall through is only for non fp types. Yes I see now... a quite complicated way to express the c

Re: Suspicious expand_complex_division() in tree-comlpex.c

2007-01-04 Thread Andrew Pinski
> > Peeking at the implementation of the expand_complex_division() function > inside tree-complex.c I have compared the conditions for satisfying > the -fcx-limited-range and flag_isoc99 > condition. And thus I have some questions about the logics of it. > In esp. the following lines (starting a

Suspicious expand_complex_division() in tree-comlpex.c

2007-01-04 Thread Marcin Dalecki
Peeking at the implementation of the expand_complex_division() function inside tree-complex.c I have compared the conditions for satisfying the -fcx-limited-range and flag_isoc99 condition. And thus I have some questions about the logics of it. In esp. the following lines (starting at 1222 insi