This is an archive of the discontinued LLVM Phabricator instance.

AMDGPU: Correctly lower llvm.exp2.f32
ClosedPublic

Authored by arsenm on Jun 15 2023, 7:09 AM.

Details

Reviewers
foad
rampitec
jhuber6
cdevadas
Pierre-vh
b-sumner
Group Reviewers
Restricted Project
Summary

Previously this did a fast math expansion only.

Diff Detail

Event Timeline

arsenm created this revision.Jun 15 2023, 7:09 AM
Herald added a project: Restricted Project. · View Herald TranscriptJun 15 2023, 7:09 AM
arsenm requested review of this revision.Jun 15 2023, 7:09 AM
Herald added a project: Restricted Project. · View Herald TranscriptJun 15 2023, 7:09 AM
Herald added a subscriber: wdng. · View Herald Transcript
rampitec added inline comments.
llvm/docs/AMDGPUUsage.rst
976

Typo 0.51ULP?

arsenm added inline comments.Jun 21 2023, 2:43 PM
llvm/docs/AMDGPUUsage.rst
976

If it's a typo it's a typo in the ISA manual

rampitec accepted this revision.Jul 5 2023, 12:43 PM
This revision is now accepted and ready to land.Jul 5 2023, 12:43 PM