RKSimon added a comment.

In https://reviews.llvm.org/D28018#639765, @GGanesh wrote:

> Fallback to CK_BTVER1 is ok but not to CK_BTVER2. This is not possible 
> because of the partial YMM writes. They have different behavior for znver1 
> with AVX and their legacy SIMD counterparts. So, as of now leaving them to 
> alphabetical order.


That makes sense for the llvm patch, but for this clang patch in 
X86TargetInfo::initFeatureMap we don't need to handle those behaviour features 
just the instruction set features which we can fall though.


https://reviews.llvm.org/D28018



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

Reply via email to