[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