[PATCH] D120665: [AArch64] Fix the TuneExynosM4 entry in lib/Target/AArch64/AArch64.td

Dave Green via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 28 11:39:28 PST 2022


dmgreen added inline comments.


================
Comment at: llvm/lib/Target/AArch64/AArch64.td:821
 
-def TuneExynosM4 : SubtargetFeature<"exynosm3", "ARMProcFamily", "ExynosM3",
-                                    "Samsung Exynos-M3 processors",
+def TuneExynosM4 : SubtargetFeature<"exynosm4", "ARMProcFamily", "ExynosM3",
+                                    "Samsung Exynos-M4 processors",
----------------
sdesmalen wrote:
> should this be ExynosM4 as well then?
It wasn't in the original code. They shared the ExynosM3 ProcFamily. Adding it doesn't add a huge amount unless there is some reason to distinguish the two.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D120665/new/

https://reviews.llvm.org/D120665



More information about the llvm-commits mailing list