[PATCH] D28018: AMD family 17h (znver1) enablement

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 9 08:51:47 PST 2017


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





More information about the llvm-commits mailing list