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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to