[PATCH] D10724: Bump X86 Darwin MaxVectorAlign to 64, for AVX512.

Ahmed Bougacha ahmed.bougacha at gmail.com
Wed Jul 15 08:30:15 PDT 2015


ab added a comment.

In http://reviews.llvm.org/D10724#194990, @rjmccall wrote:

> Should we conditionalize this on whether AVX (for 32-byte) and AVX-512 (for 64-byte) are enabled?  I'm willing to accept that we shouldn't; just want to hear your thoughts.


I think that would make sense.

I thought I saw a justification for unconditionally using 256 in the original AVX patch, but now that I look again I don't think there is one, and I can't think of one.


http://reviews.llvm.org/D10724







More information about the cfe-commits mailing list