This is an archive of the discontinued LLVM Phabricator instance.

[NVPTX] Added LDU/LDG intrinsics for f16.
ClosedPublic

Authored by tra on Mar 1 2017, 3:01 PM.

Diff Detail

Repository
rL LLVM

Event Timeline

tra created this revision.Mar 1 2017, 3:01 PM
jlebar edited edge metadata.Mar 1 2017, 3:03 PM

Do we not have tests for this? On the one hand it's kind of simple enough that it's not worth testing, but otoh it would be pretty annoying if it broke...

tra updated this revision to Diff 90254.Mar 1 2017, 4:26 PM

Added one more missing load instruction variant for f16.

tra added a comment.Mar 1 2017, 4:39 PM

I'll add proper tests along with the upcoming patch for lowering f16x2 ldg/ldu.

jlebar accepted this revision.Mar 1 2017, 4:42 PM
This revision is now accepted and ready to land.Mar 1 2017, 4:42 PM
This revision was automatically updated to reflect the committed changes.