This is an archive of the discontinued LLVM Phabricator instance.

AMDGPU: Remove unnecessary ands when f16 is legal
ClosedPublic

Authored by arsenm on Mar 17 2017, 12:42 PM.

Details

Reviewers
kzhuravl
Summary

Add a new node to act as a fancy bitcast from f16 operations to
i32 that implicitly zero the high 16-bits of the result.

Alternatively could try making v2f16 legal and canonicalizing
on build_vectors.

Diff Detail

Event Timeline

arsenm created this revision.Mar 17 2017, 12:42 PM
arsenm updated this revision to Diff 92183.Mar 17 2017, 12:50 PM

Remove dead pattern

t-tye added a subscriber: t-tye.Mar 22 2017, 6:40 PM
tony-tye removed a subscriber: tony-tye.Mar 22 2017, 6:47 PM
This revision is now accepted and ready to land.Mar 24 2017, 11:49 AM
arsenm closed this revision.Mar 31 2017, 1:05 PM

r299246