Adds support for the new Exynos-M1 ARMv8-A core.
|255 ↗||(On Diff #43353)|
I think that might confuse people thinking it is a ARMv8-M core rather than an ARMv8-A. Might be better to change it to ProcExynosM1 or something similar.
LGTM. But since this is my first review, I would like somebody else to sign off on this as well.
By the way: when putting a review in Phabricator, please use please use "-U999" so that the context around the change is available. Otherwise it might requires the reviewers to look through there local copies of the sources to understand the changes.
It might be my first review, but let me just sign off on this one as it LGTM. The more senior guys are either in the new-year holiday mode or just too busy after the holidays. Sorry for the delay.
No reservations from my side, maybe Tim or James have a comment? Though this could also be post-commit. LGTM, too.
I wish the AArch64 back-end would move to TargetParser, but that's another discussion entirely.