This is an archive of the discontinued LLVM Phabricator instance.

[PowerPC] Add DFP test instruction definitions and MC tests
ClosedPublic

Authored by kamaub on Jul 6 2023, 6:59 AM.

Details

Summary

Adding the td definitions, and their associated assembly and disassembly
test cases for the decimal floating point test instructions defined in
section 5.6.3 of ISA 3.1

Diff Detail

Event Timeline

kamaub created this revision.Jul 6 2023, 6:59 AM
Herald added a project: Restricted Project. · View Herald TranscriptJul 6 2023, 6:59 AM
kamaub requested review of this revision.Jul 6 2023, 6:59 AM
Herald added a project: Restricted Project. · View Herald TranscriptJul 6 2023, 6:59 AM
kamaub updated this revision to Diff 538177.Jul 7 2023, 9:41 AM

Removing the "mayRaiseFPException" predicate as it does not apply to these instructions.

amyk accepted this revision as: amyk.Jul 11 2023, 10:17 AM

I think since we discussed offline that the Z22 forms extended opcode should start at 22, this LGTM.

llvm/lib/Target/PowerPC/PPCInstrFormats.td
1225

nit: Indentation

2191

nit: Indentation

This revision is now accepted and ready to land.Jul 11 2023, 10:17 AM
lei added inline comments.Jul 12 2023, 1:52 PM
llvm/lib/Target/PowerPC/PPCInstrDFP.td
132

nit: indent the following 2 lines 1 more space

144

nit: the 2nd line for this and the next 3 instr require 1 more space in indent

160

nit: indent

lei accepted this revision as: lei.Jul 12 2023, 1:55 PM

LGTM other then some nits.
Thx.

This revision was landed with ongoing or failed builds.Jul 14 2023, 11:00 AM
This revision was automatically updated to reflect the committed changes.
kamaub marked 5 inline comments as done.