[PATCH] D127812: [AArch64] FMV support and necessary target features dependencies.

[Pavel Iliin via Phabricator via cfe-commits]

ilinpv added inline comments.


================
Comment at: llvm/include/llvm/Support/AArch64TargetParser.def:115
+AARCH64_ARCH_EXT_NAME("rdm",           AArch64::AEK_RDM,         "+rdm",       
   "-rdm",         \
+RDM,          "+rdm,+fp-armv8,+neon,+jsconv,+complxnum",                       
         70)
+AARCH64_ARCH_EXT_NAME("crypto",        AArch64::AEK_CRYPTO,      "+crypto",    
   "-crypto",      \
----------------
dmgreen wrote:
> Should RDM be dependant on +jsconv,+complxnum?
I suppose RDM implies simd (+neon) which implies +jsconv,+complxnum


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D127812

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