Page MenuHomePhabricator

[VE] Support f128
ClosedPublic

Authored by kaz7 on Aug 16 2020, 4:22 AM.

Details

Summary

Support f128 using VE instructions. Update regression tests.
I've noticed there is no load or store i128 test, so I add them too.

Diff Detail

Event Timeline

kaz7 created this revision.Aug 16 2020, 4:22 AM
kaz7 requested review of this revision.Aug 16 2020, 4:22 AM

Please look into the lint comments. Otw, LGTM.

llvm/lib/Target/VE/VEInstrInfo.td
901

unrelated change

kaz7 planned changes to this revision.Aug 17 2020, 12:36 AM

Thanks. I'll update source code following your suggestion and lit results.

llvm/lib/Target/VE/VEInstrInfo.td
901

Thanks. I'll remove this.

kaz7 updated this revision to Diff 285928.Aug 17 2020, 12:51 AM

Update code.

simoll accepted this revision.Aug 17 2020, 1:09 AM
This revision is now accepted and ready to land.Aug 17 2020, 1:09 AM
This revision was landed with ongoing or failed builds.Aug 17 2020, 1:27 AM
Closed by commit rG40f1e7e80401: [VE] Support f128 (authored by kaz7). · Explain Why
This revision was automatically updated to reflect the committed changes.