[PATCH] D65800: [LLVM][Alignment] Introduce Alignment In MachineFrameInfo

Guillaume Chatelet via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 6 08:31:27 PDT 2019


gchatelet added a comment.

All the tests are passing but I'm not entirely sure of the validity of this patch.
`Alignment`, `StackAlignment` seems to be always defined alignment (aka. `Align`) but I'm not so sure of `MaxAlignment` and `LocalFrameMaxAlign`.
`Hexagon/HexagonFrameLowering.cpp` seem to rely on the fact that `LocalFrameMaxAlign` is undefined.
I'm not sure about `MaxAlignment`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D65800





More information about the llvm-commits mailing list