[Patch, Fortran] PR fortran/46794: Fix ICE with powers of integers
Daniel Kraft
d@domob.eu
Thu Dec 9 16:16:00 GMT 2010
Tobias Burnus wrote:
> Dear Daniel,
>
> Daniel Kraft wrote:
>> I do not entirely like the way this is done in the patch (with the two
>> variables and "special casing"), but don't see a better implementation
>> -- what do you think?
>
> I think it is OK - at least I have trouble finding a better solution.
>
>> Regression-tested on x86_64-unknown-linux-gnu without failures --
>> though the run somehow looked strange to me (on the compile-farm);
>> I'll try again to be sure. Ok for trunk?
>
> OK for the trunk. Can you check whether one needs to likewise for the
> 4.5 and 4.4 branch? (I think one should check on source level - the
> verify_tree might not always catch it. For some reasons, it ICEs here
> with 4.4 and 4.6 but not with 4.5; however, I think that's rather by
> chance and not because of a proper casting.)
Committed the attached (patch adapted to 4.4 including fix of test-case)
as rev. 167644 to 4.4 branch. When 4.5 is open again, I will fix the
test-case there, too, and then close the PR.
Yours,
Daniel
--
http://www.pro-vegan.info/
--
Done: Arc-Bar-Cav-Kni-Ran-Rog-Sam-Tou-Val-Wiz
To go: Hea-Mon-Pri
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: patch.changelog
URL: <http://gcc.gnu.org/pipermail/gcc-patches/attachments/20101209/749116d7/attachment.ksh>
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: patch.diff
URL: <http://gcc.gnu.org/pipermail/gcc-patches/attachments/20101209/749116d7/attachment-0001.ksh>
More information about the Gcc-patches
mailing list