This is an archive of the discontinued LLVM Phabricator instance.

[X86] Remove the cvtuqq2ps256/cvtqq2ps256 mask builtins. Replace with uitofp/sitofp and select.
ClosedPublic

Authored by craig.topper on Jan 19 2019, 11:47 AM.

Diff Detail

Repository
rL LLVM

Event Timeline

craig.topper created this revision.Jan 19 2019, 11:47 AM
RKSimon added a comment.EditedJan 19 2019, 1:30 PM

Just to confirm, this isn't driven by RC in any way?

It uses current rounding mode for inexact conversions. cvtsi642ss should do the same. As does (u)dq2ps and (u)qq2pd. I think we use sitofp/uitofp for some lengths of those already.

RKSimon accepted this revision.Jan 20 2019, 2:33 AM

OK - LGTM, thanks

This revision is now accepted and ready to land.Jan 20 2019, 2:33 AM
This revision was automatically updated to reflect the committed changes.