[PATCH] D63810: Change the default arm-linux-gnueabihf target to armv7

[Peter Smith via Phabricator via llvm-commits]

peter.smith added a comment.

The arm1176jfz-s is what is used on the first revision of the Raspberry Pi and I think some of the later reduced size versions. This corresponds to the first mass market ARM linux distribution incorporating HF. Changing the default to ARMv7 would mean that people using clang for a Raspberry Pi using the default would end up with programs that likely would not run due to use of Arm v7 instructions. It is difficult to know how many people actually care about this as I suspect that Distros that still support the first revision of Raspberry Pi can change the default back. I think Sjoerd is right and a question on llvm-dev to canvas opinion would be a good idea.

If we were to go for Armv7 I also think that Cortex-a7 may not be the best choice. This is not the entry point for Armv7 as it includes the virtualization features but also looks like it has some code-generation tweaks to account for slow performance of certain instructions, this could result in slower performance for a later CPU. I'd prefer something equivalent to -march=armv7a -mcpu=generic.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D63810