[PATCH] D49720: [ARM] Fix over-alignment in arguments that are HA of 128-bit vectors

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 25 12:53:27 PDT 2018


efriedma added inline comments.


================
Comment at: lib/Target/ARM/ARMCallingConv.h:279
     // we'll be allocating a bunch of i32 slots).
-    Align = Size;
+    Align = std::min(Align, Size);
   }
----------------
Could you hoist the `std::min(Align, Size)` out of the loop so it's clear it isn't changing every iteration?


Repository:
  rL LLVM

https://reviews.llvm.org/D49720





More information about the llvm-commits mailing list