This is an archive of the discontinued LLVM Phabricator instance.

[flang] Don't force SET_EXPONENT(I=...) argument to integer(4)
ClosedPublic

Authored by klausler on Oct 4 2022, 2:54 PM.

Details

Summary

The implementation of the folding code for SET_EXPONENT() was written
in such a fashion as to convert the I= actual argument value to a 32-bit
integer. Which is usually not a problem, but it's not always correct
and a test case ran into trouble with it. Fix to allow any kind of
INTEGER without conversion.

Diff Detail

Event Timeline

klausler created this revision.Oct 4 2022, 2:54 PM
Herald added a project: Restricted Project. · View Herald TranscriptOct 4 2022, 2:54 PM
Herald added a subscriber: jdoerfert. · View Herald Transcript
klausler requested review of this revision.Oct 4 2022, 2:54 PM
jeanPerier accepted this revision.Oct 5 2022, 12:16 AM
This revision is now accepted and ready to land.Oct 5 2022, 12:16 AM
klausler updated this revision to Diff 465789.Oct 6 2022, 11:05 AM

Properly handle underflow case in SET_EXPONENT.

This revision was landed with ongoing or failed builds.Oct 6 2022, 11:20 AM
This revision was automatically updated to reflect the committed changes.