[clang] [llvm] [PowerPC][AIX] Specify correct ABI alignment for double (PR #144673)
Nikita Popov via cfe-commits
cfe-commits at lists.llvm.org
Thu Sep 25 00:51:38 PDT 2025
================
@@ -378,15 +378,6 @@ static void checkDataLayoutConsistency(const TargetInfo &Target,
Check("bfloat", llvm::Type::getBFloatTy(Context), Target.BFloat16Align);
Check("float", llvm::Type::getFloatingPointTy(Context, *Target.FloatFormat),
Target.FloatAlign);
- // FIXME: AIX specifies wrong double alignment in DataLayout
- if (!Triple.isOSAIX()) {
- Check("double",
- llvm::Type::getFloatingPointTy(Context, *Target.DoubleFormat),
- Target.DoubleAlign);
- Check("long double",
- llvm::Type::getFloatingPointTy(Context, *Target.LongDoubleFormat),
- Target.LongDoubleAlign);
----------------
nikic wrote:
Oops, indeed. I've made the checks unconditional now.
https://github.com/llvm/llvm-project/pull/144673
More information about the cfe-commits
mailing list