[clang] [clang] Limit alignment for emitted vectors (PR #98629)
Phoebe Wang via cfe-commits
cfe-commits at lists.llvm.org
Mon Jul 15 07:44:33 PDT 2024
================
@@ -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
More information about the cfe-commits
mailing list