This is an archive of the discontinued LLVM Phabricator instance.

AMDGPU: Stop trying to specially handle vector stores in printf lowering
ClosedPublic

Authored by arsenm on Jan 6 2023, 3:32 PM.

Details

Reviewers
sameerds
vikramRH
Group Reviewers
Restricted Project
Summary

This was broken for 1 element vectors and trying to create invalid
casts. We can directly store any type just fine, so don't bother with
this buggy conversion logic.

Diff Detail

Event Timeline

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