- Add M68k as new Clang target
- Add new attribute to support M68k's ISR (Interrupt Service Routine)
Looking forward to see m68k support (and hopefully sega genesis toolchain support someday)!
|73 ↗||(On Diff #302187)|
This doesn't seem to match the coding style, clang-format to the rescue.
|89 ↗||(On Diff #302187)|
Can you make all the defineMacro inline within the cases? Clarity seems more useful here.
|35 ↗||(On Diff #302187)|
Can you remove the comment and add the enum back whenever you introduce CK_68060 support?
- Addressed all the feedbacks
- Fixed minor issues that would retrieve the wrong TargetCodeGenInfo instance
currently we don't have any plan for that
|123 ↗||(On Diff #302187)|
stack pointer is aliased to a7, but I'll add pc into the list.
Seems to work after I fixed a minor issue in getting the correct TargetCodeGenInfo. Will remove this comment.
|38–50 ↗||(On Diff #309461)|
If we're in SysV psABI land, then the stack is 32-bit aligned.
If we're in actual ABI used by everyone out there, i.e. GCC's default, then it's only 16-bit aligned, and your integer types also have the wrong alignment, so you will get ABI incompatibility with GCC-built binaries as provided by distributions like Debian.
|63 ↗||(On Diff #309461)|
GCC's -mcpu excludes any M prefix and is just the number.
|77–79 ↗||(On Diff #309461)|
Where are these coming from? GCC only defines __m68k__.