[PATCH] D18086: Fix default processor name for armv6k.

[Renato Golin via Phabricator via llvm-commits]

rengolin added inline comments.


================
Comment at: include/llvm/Support/ARMTargetParser.def:182
 ARM_CPU_NAME("arm1136jz-s", AK_ARMV6, FK_NONE, false, AEK_NONE)
-ARM_CPU_NAME("arm1176j-s", AK_ARMV6K, FK_NONE, true, AEK_NONE)
+ARM_CPU_NAME("arm1176jz-s", AK_ARMV6K, FK_NONE, false, AEK_NONE)
+ARM_CPU_NAME("arm1176jzf-s", AK_ARMV6K, FK_VFPV2, true, AEK_NONE)
----------------
srhines wrote:
> This uses the same name as line 184, which seems like a bug to me.
I think that's what @labrinea was talking about.

Can anyone from Arm confirm what "j-s" really is, if it is anything at all?

I had so many conversations about this in the past and all of them got me even more confused than before.


https://reviews.llvm.org/D18086