[PATCH] D77984: Make IRBuilder automatically set alignment on load/store/alloca.

Mon Apr 13 21:42:43 PDT 2020

bondhugula added a comment.

In D77984#1979644 <https://reviews.llvm.org/D77984#1979644>, @bondhugula wrote:

> This revision has broken an MLIR test case that test conversion to LLVM IR (test/Target/llvmir.mlir - an llvm.alloca in MLIR with its alignment attribute set to 0 now converts into an LLVM IR alloca with alignment automatically set to 4 - I think the fix is trivial; the check there is expecting to see no align attribute attached to the resulting alloca). Please build with -DLLVM_ENABLE_PROJECTS=mlir to reproduce, but revisions submitted via arcanist will already show the failed test cases.

I've gone ahead and fixed it in 3c30e17346a324029bbb59abee9f7e6a9265dfc7.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D77984