Re: [PATCH] D12243: Properly provide alignment of 'byval' arguments down to llvm.

2015-08-21 Thread James Y Knight via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL245719: Properly provide alignment of 'byval' arguments down to llvm. (authored by jyknight). Changed prior to commit: http://reviews.llvm.org/D12243?vs=32839&id=32847#toc Repository: rL LLVM http:/

Re: [PATCH] D12243: Properly provide alignment of 'byval' arguments down to llvm.

2015-08-21 Thread David Majnemer via cfe-commits
majnemer accepted this revision. majnemer added a comment. This revision is now accepted and ready to land. LGTM http://reviews.llvm.org/D12243 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cf

[PATCH] D12243: Properly provide alignment of 'byval' arguments down to llvm.

2015-08-21 Thread James Y Knight via cfe-commits
jyknight created this revision. jyknight added a reviewer: majnemer. jyknight added a subscriber: cfe-commits. This is important in the case that the LLVM-inferred llvm-struct alignment is not the same as the clang-known C-struct alignment. http://reviews.llvm.org/D12243 Files: lib/CodeGen/CGC