This is an archive of the discontinued LLVM Phabricator instance.

[M68k] Add subtarget features for M68881/2 FPU
ClosedPublic

Authored by myhsu on Apr 3 2023, 3:58 PM.

Details

Summary

Note that technically both M68000/010 can use M68881, despite the fact that usually only M68020 and newer ISAs are equipped with M68881/2. M68040 and newer processors have builtin M68882.

Diff Detail

Event Timeline

myhsu created this revision.Apr 3 2023, 3:58 PM
Herald added a project: Restricted Project. · View Herald TranscriptApr 3 2023, 3:58 PM
Herald added a subscriber: hiraditya. · View Herald Transcript
myhsu requested review of this revision.Apr 3 2023, 3:58 PM
Herald added a project: Restricted Project. · View Herald TranscriptApr 3 2023, 3:58 PM
0x59616e added inline comments.Apr 5 2023, 6:46 PM
llvm/lib/Target/M68k/M68k.td
47

nit: need a space to align with the current style.

0x59616e accepted this revision.Apr 5 2023, 7:50 PM
This revision is now accepted and ready to land.Apr 5 2023, 7:50 PM
myhsu updated this revision to Diff 511452.Apr 6 2023, 9:37 AM
myhsu marked an inline comment as done.

minor formatting.

This revision was automatically updated to reflect the committed changes.