This is an archive of the discontinued LLVM Phabricator instance.

[AMDGPU] Make v16i16/v16f16 legal
ClosedPublic

Authored by piotr on Jun 29 2022, 4:48 PM.

Details

Summary

There are upcoming intrinsics to use the new types.

Diff Detail

Event Timeline

piotr created this revision.Jun 29 2022, 4:48 PM
Herald added a project: Restricted Project. · View Herald TranscriptJun 29 2022, 4:48 PM
piotr requested review of this revision.Jun 29 2022, 4:48 PM
Herald added a project: Restricted Project. · View Herald TranscriptJun 29 2022, 4:48 PM
arsenm added inline comments.Jun 29 2022, 5:28 PM
llvm/test/Analysis/CostModel/AMDGPU/arith-usat.ll
118–119

Don’t understand why this happened but it’s not really important

llvm/test/CodeGen/AMDGPU/inlineasm-illegal-type.ll
23–25

Should retain this in one of the working asm tests

piotr updated this revision to Diff 441292.Jun 30 2022, 12:57 AM

Added a new inline asm test file with converted test, plus a few others.

This revision is now accepted and ready to land.Jun 30 2022, 9:40 AM
This revision was landed with ongoing or failed builds.Jun 30 2022, 2:10 PM
This revision was automatically updated to reflect the committed changes.