[PATCH] D12337: [Codegen] Ensure stack is properly aligned for call argument initialization

Jeroen Ketema via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 1 17:37:57 PDT 2015


jketema marked 6 inline comments as done.
jketema added a comment.

Hi Reid,

Could you have another look at this? After having had a renewed look at the code, `getNextStackOffset` still seems the correct place for this change to me for the reasons I provided in the inline comments. However, if you think this really belongs somewhere else, would you be able to point me in the right direction?

Thanks,

Jeroen


http://reviews.llvm.org/D12337





More information about the llvm-commits mailing list