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

Simon Pilgrim via Phabricator via cfe-commits cfe-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.


More information about the cfe-commits mailing list