[PATCH] D83079: [clang] Default target features implied by `-march` on AArch64.

[Sjoerd Meijer via Phabricator via cfe-commits]

SjoerdMeijer added a comment.

I haven't looked into details of these arch extensions yet, will do that tomorrow, but I don't think there's any disagreement, i.e., these options and their behaviour are synchronised with the GCC community.  Thus, it's better if you remove this wording from both description and comments in the source code. If there is divergence in behaviour, then that is probably be an oversight somewhere. I am also pretty sure that this 8.5 stuff is implemented, not sure if that is upstreamed. And also, I think we need to be more specific than "Default target features implied by `-march` on AArch64", because this is a big topic, which is not addressed in this patch.  This patch is only about a few architecture extension. I am also for example surprised to see bfloat there, as I saw quite some activity in this area.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D83079