This is an archive of the discontinued LLVM Phabricator instance.

AMDGPU: Use pown instead of pow if known integral
ClosedPublic

Authored by arsenm on Aug 28 2023, 8:28 AM.

Details

Reviewers
rampitec
jmmartinez
Group Reviewers
Restricted Project

Diff Detail

Event Timeline

arsenm created this revision.Aug 28 2023, 8:28 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 28 2023, 8:28 AM
arsenm requested review of this revision.Aug 28 2023, 8:28 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 28 2023, 8:28 AM
Herald added a subscriber: wdng. · View Herald Transcript
rampitec added inline comments.Aug 28 2023, 10:40 AM
llvm/lib/Target/AMDGPU/AMDGPULibFunc.cpp
909

I8 if signed, not U8.

arsenm updated this revision to Diff 554861.Aug 30 2023, 4:36 PM

fix swapped i8/u8

arsenm marked an inline comment as done.Aug 30 2023, 4:37 PM
rampitec accepted this revision.Aug 31 2023, 12:16 AM
This revision is now accepted and ready to land.Aug 31 2023, 12:16 AM