This is an archive of the discontinued LLVM Phabricator instance.

[flang] Improve warning message
ClosedPublic

Authored by klausler on Oct 29 2022, 5:27 PM.

Details

Summary

When a floating-point exception occurs while folding an exponentiation
operation with a host library at compilation time, the warning message
should not say that it was an "intrinsic function".

Diff Detail

Event Timeline

klausler created this revision.Oct 29 2022, 5:27 PM
Herald added a project: Restricted Project. · View Herald TranscriptOct 29 2022, 5:27 PM
Herald added a subscriber: jdoerfert. · View Herald Transcript
klausler requested review of this revision.Oct 29 2022, 5:27 PM
klausler updated this revision to Diff 471797.Oct 29 2022, 5:29 PM

Correct the test results for Evaluate/folding04.f90.

Evaluate/folding04.f90 still seems to fail after your patch update. It seems llvm buildbots run yield "overflow" errors and not "division by zero" errors in clog((0., 0.)) and others. Could it be a difference in what the underlying runtime math library is signaling when folding ?

Evaluate/folding04.f90 still seems to fail after your patch update. It seems llvm buildbots run yield "overflow" errors and not "division by zero" errors in clog((0., 0.)) and others. Could it be a difference in what the underlying runtime math library is signaling when folding ?

I can make the test specific to x86, or I can make the error messages more generic ("floating-point exception") just to make the test pass.

klausler updated this revision to Diff 472106.Oct 31 2022, 12:47 PM

Adjust test for portability.

This revision is now accepted and ready to land.Nov 1 2022, 1:51 AM
This revision was automatically updated to reflect the committed changes.