This is an archive of the discontinued LLVM Phabricator instance.

AMDGPU: Fix enabling denormals by default on pre-VI targets
ClosedPublic

Authored by arsenm on Aug 7 2018, 2:46 AM.

Details

Reviewers
b-sumner
kzhuravl
Summary

Fast FMAF is not a sufficient condition to enable denormals.
Before VI, enabling denormals caused F32 instructions to
run at F64 speeds.

Diff Detail

Event Timeline

arsenm created this revision.Aug 7 2018, 2:46 AM

This approach seems fine to me.

kzhuravl accepted this revision.Aug 8 2018, 6:04 AM

LGTM

This revision is now accepted and ready to land.Aug 8 2018, 6:04 AM
arsenm closed this revision.Aug 8 2018, 10:49 AM

r339278