This is an archive of the discontinued LLVM Phabricator instance.

[x86] Solve the problem caused by strict and non-strict FP-TO-S/UINT sharing CVTTP2SI/CVTTP2UI nodes
AbandonedPublic

Authored by LiuChen3 on Dec 19 2019, 12:40 AM.

Details

Summary

Before this patch, It will prevent emitting masked conversion instruction and emit an unmasked instruction and a masked move instruction.
With this patch, non-strict FP_TO_S/UINT will emit one masked conversion instruction.

Diff Detail

Event Timeline

LiuChen3 created this revision.Dec 19 2019, 12:40 AM
Herald added a project: Restricted Project. · View Herald TranscriptDec 19 2019, 12:40 AM
LiuChen3 updated this revision to Diff 234677.Dec 19 2019, 1:27 AM

Simplified test cases

LiuChen3 abandoned this revision.Dec 23 2019, 9:43 PM

Since D71850 has separated strict and non-strict node.