[PATCH] D133338: [clang][PowerPC] PPC64 VAArg use coerced integer type for direct aggregate fits in register

Ulrich Weigand via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Sep 8 07:10:17 PDT 2022


uweigand added inline comments.


================
Comment at: clang/lib/CodeGen/TargetInfo.cpp:5471
+        if (CoerceTy->isIntegerTy() && CoerceTy->getIntegerBitWidth() < GPRBits)
+          ForceRightAdjust = true;
+  }
----------------
Are all these checks really necessary here?  This seems to duplicate the checks that are already in `emitVoidPtrVAArg` ...    Can't we simply always pass `true` for `ForceRightAdjust` on PowerPC?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D133338/new/

https://reviews.llvm.org/D133338



More information about the cfe-commits mailing list