[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