This is an archive of the discontinued LLVM Phabricator instance.

[GlobalISel][AArch64] Legalize vector G_FMA + fix G_FMA's register bank
ClosedPublic

Authored by paquette on Apr 19 2019, 9:56 AM.

Details

Reviewers
aemerson
paquette
Summary

This is legalized the same way as G_FCEIL, etc. Just add it to the existing legalizer rule.

Also, add the missing switch statement in AArch64RegisterBankInfo.cpp to make sure G_FMA is always on an FPR.

Update arm64-vfloatintrinsics.ll, add a legalizer test for G_FMA, and add a regbankselect test for G_FMA.

Diff Detail

Event Timeline

paquette created this revision.Apr 19 2019, 9:56 AM
paquette accepted this revision.Apr 23 2019, 4:22 PM

was told to YOLO trivial patches like these offline

r359015

This revision is now accepted and ready to land.Apr 23 2019, 4:22 PM
paquette closed this revision.Apr 23 2019, 4:23 PM