[PATCH] D67852: [Alignment] Get DataLayout::StackAlignment as Align

Clement Courbet via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Sep 22 23:55:29 PDT 2019


courbet added a comment.

Can you please explain why the switch from `MaybeAlign` to `Align` is fine ? `ARMTargetLowering::getABIAlignmentForCallingConv` can switch from non-zero to 0 it seems.

I see another call in the WebAsm backend.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D67852





More information about the llvm-commits mailing list