This is an archive of the discontinued LLVM Phabricator instance.

Rename CONVERT_RNDSAT td name to better match it
AbandonedPublic

Authored by arsenm on Jan 14 2015, 2:59 PM.

Details

Reviewers
None
Summary

I want to define an instruction called cvt, which was
surprisingly already taken by this. This node is not supposed
to be directly used, and has a somewhat surprising name. Rename it
to make it more obvious what it is, and more appropriate for
a private node.

Diff Detail

Event Timeline

arsenm updated this revision to Diff 18189.Jan 14 2015, 2:59 PM
arsenm retitled this revision from to Rename CONVERT_RNDSAT td name to better match it .
arsenm updated this object.
arsenm edited the test plan for this revision. (Show Details)
arsenm added a subscriber: Unknown Object (MLST).
ab added a subscriber: ab.Jan 14 2015, 3:16 PM

So.. I've been wondering about this for a while: these (and the
associated ISD opcode) don't seem to be referenced by any target, and
the comment says the opcode is pretty much deprecated. Maybe we should
just remove them completely?

  • Ahmed
arsenm abandoned this revision.Aug 11 2016, 11:27 AM