This is an archive of the discontinued LLVM Phabricator instance.

AMDGPU: Add llvm.amdgcn.exp2 intrinsic
ClosedPublic

Authored by arsenm on Jun 14 2023, 5:34 AM.

Details

Reviewers
foad
rampitec
yaxunl
Pierre-vh
cdevadas
Group Reviewers
Restricted Project
Summary

Provide direct access to v_exp_f32 and v_exp_f16, so we can start
correctly lowering the generic exp intrinsics.

Unfortunately have to break from the usual naming convention of
matching the instruction name and stripping the v_ prefix. exp is
already taken by the export intrinsic. On the clang builtin side, we
have a choice of maintaining the convention to the instruction name,
or following the intrinsic name.

Diff Detail

Event Timeline

arsenm created this revision.Jun 14 2023, 5:34 AM
Herald added a project: Restricted Project. · View Herald TranscriptJun 14 2023, 5:34 AM
arsenm requested review of this revision.Jun 14 2023, 5:34 AM
Herald added a project: Restricted Project. · View Herald TranscriptJun 14 2023, 5:34 AM
Herald added a subscriber: wdng. · View Herald Transcript
arsenm updated this revision to Diff 531300.Jun 14 2023, 6:25 AM

Fix patch split

arsenm updated this revision to Diff 531301.Jun 14 2023, 6:32 AM
foad added inline comments.Jun 14 2023, 6:49 AM
llvm/lib/Target/AMDGPU/AMDGPUISelLowering.h
463

Is this used anywhere?

arsenm added inline comments.Jun 14 2023, 7:09 AM
llvm/lib/Target/AMDGPU/AMDGPUISelLowering.h
463

Will be in the next patch

arsenm updated this revision to Diff 531315.Jun 14 2023, 7:09 AM

Release notes

foad accepted this revision.Jun 14 2023, 7:42 AM
This revision is now accepted and ready to land.Jun 14 2023, 7:42 AM