This is an archive of the discontinued LLVM Phabricator instance.

AMDGPU: Don't assert on printf of half
ClosedPublic

Authored by arsenm on Jan 6 2023, 4:05 PM.

Details

Reviewers
sameerds
vikramRH
Group Reviewers
Restricted Project
Summary

The comment says fields should be 4-byte aligned, so just pass through
after conversion to integer. The conformance test lacks any testing of
half.

Diff Detail

Event Timeline

arsenm created this revision.Jan 6 2023, 4:05 PM
Herald added a project: Restricted Project. · View Herald TranscriptJan 6 2023, 4:05 PM
arsenm requested review of this revision.Jan 6 2023, 4:05 PM
Herald added a project: Restricted Project. · View Herald TranscriptJan 6 2023, 4:05 PM
Herald added a subscriber: wdng. · View Herald Transcript
sameerds accepted this revision.Jan 9 2023, 2:47 AM
This revision is now accepted and ready to land.Jan 9 2023, 2:47 AM