Page MenuHomePhabricator

AMDGPU: Assume f32 denormals are enabled by default
ClosedPublic

Authored by arsenm on Dec 11 2019, 6:43 AM.

Details

Reviewers
rampitec
b-sumner
Summary

This will likely introduce catastrophic performance regressions, but
should be correct. A follow-up change will switch the default based on
the calling convention / function attributes which should avoid the
regressions.

Diff Detail

Event Timeline

arsenm created this revision.Dec 11 2019, 6:43 AM
Herald added a project: Restricted Project. · View Herald TranscriptDec 11 2019, 6:43 AM
rampitec added inline comments.
llvm/docs/ReleaseNotes.rst
154

Are these changes to FE already done?

arsenm marked an inline comment as done.Jan 6 2020, 8:29 AM
arsenm added inline comments.
llvm/docs/ReleaseNotes.rst
154

Yes, for clang

I am OK with proceeding here.

This revision is now accepted and ready to land.Jan 6 2020, 11:44 AM