This is an archive of the discontinued LLVM Phabricator instance.

AMDGPU/GlobalISel: Select G_FABS/G_FNEG
ClosedPublic

Authored by arsenm on Sep 1 2019, 4:39 PM.

Details

Summary

f64 doesn't work yet because tablegen currently doesn't handlde
REG_SEQUENCE.

This does regress some multi use VALU fneg cases since now the
immediate remains in an SGPR, and more moves are used for legalizing
the xor. This is a SIFixSGPRCopies deficiency.

Diff Detail

Event Timeline

arsenm created this revision.Sep 1 2019, 4:39 PM
kerbowa accepted this revision.Sep 10 2019, 9:50 AM

LGTM

This revision is now accepted and ready to land.Sep 10 2019, 9:50 AM
arsenm closed this revision.Sep 10 2019, 10:18 AM

r371540