This is an archive of the discontinued LLVM Phabricator instance.

[VE] Update floating-point arithmetic instructions
ClosedPublic

Authored by kaz7 on Apr 23 2020, 3:48 PM.

Details

Summary

Changing all mnemonic to match assembly instructions to simplify mnemonic
naming rules. This time update all floating-point arithmetic instructions.

Diff Detail

Event Timeline

kaz7 created this revision.Apr 23 2020, 3:48 PM

One comment inline. Otw, LGTM.

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

Doesn't the CVTLDr require an explicit rounding mode as in l1164f above?

kaz7 marked an inline comment as done.Apr 24 2020, 1:41 AM
kaz7 added inline comments.
llvm/lib/Target/VE/VEInstrInfo.td
1168

Yes, CVTLD (double to long) requires rounding mode. For example, CVTDL (long to double) doesn't require rounding mode, though.

simoll accepted this revision.Apr 24 2020, 2:07 AM
This revision is now accepted and ready to land.Apr 24 2020, 2:07 AM
This revision was automatically updated to reflect the committed changes.