[PATCH] D75863: [AIX] Implement by-val caller arguments in a single register

Chris Bowler via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 10 08:07:06 PDT 2020


cebowleratibm marked an inline comment as done.
cebowleratibm added inline comments.


================
Comment at: llvm/lib/Target/PowerPC/PPCISelLowering.cpp:7112
+      for (unsigned Bytes = 0; Bytes != ByValSize;) {
+        unsigned N = PowerOf2Floor(ByValSize - Bytes);
+        const MVT VT =
----------------
To reviewers, I thought I had a bug that we may choose a PowerOf2Floor that's larger than a register.  This will be a bug, however, for the time being, we're only processing ByValSize <= PtrByteSize.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75863





More information about the llvm-commits mailing list