Re: [PATCH] D16607: Implementation of PS4 ABI, round 1

2016-01-28 Thread Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL259095: Implementation of PS4 ABI, Round 1 (authored by ssrivastava). Changed prior to commit: http://reviews.llvm.org/D16607?vs=46060&id=46310#toc Repository: rL LLVM http://reviews.llvm.org/D16607

Re: [PATCH] D16607: Implementation of PS4 ABI, round 1

2016-01-28 Thread Dmitry Polukhin via cfe-commits
DmitryPolukhin added a comment. LGTM http://reviews.llvm.org/D16607 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D16607: Implementation of PS4 ABI, round 1

2016-01-27 Thread John McCall via cfe-commits
rjmccall added a comment. Seems reasonable. http://reviews.llvm.org/D16607 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D16607: Implementation of PS4 ABI, round 1

2016-01-27 Thread Paul Robinson via cfe-commits
probinson added a comment. Subscribed the reviewers of http://reviews.llvm.org/D14980, which had fixed the alignment-attribute layout bug. http://reviews.llvm.org/D16607 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/

[PATCH] D16607: Implementation of PS4 ABI, round 1

2016-01-26 Thread Sunil Srivastava via cfe-commits
Sunil_Srivastava created this revision. Sunil_Srivastava added a reviewer: rsmith. Sunil_Srivastava added a subscriber: cfe-commits. We are planning to upstream changes needed for maintaining PS4 ABI across future releases. Unlike many other targets, we need to maintain, in most cases, strict ob