================
@@ -133,6 +133,7 @@ struct TransferrableTargetInfo {
   unsigned short SuitableAlign;
   unsigned short NewAlign;
   unsigned MaxVectorAlign;
----------------
phoebewang wrote:

I'm surprised the change of alignment affects ABI. Did you just expand
```
    MaxVectorAlign =
        hasFeature("avx512f") ? 512 : hasFeature("avx") ? 256 : 128;
```
to other targets in X86.h, or together with the change here?

I assumed the former won't change return type to `set`, though align may need 
to update.

https://github.com/llvm/llvm-project/pull/98629
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to