[PATCH] D96265: [PowerPC] Change target data layout for 16-byte stack alignment

2021-03-08 Thread Ahsan Saghir via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGacce401068e7: [PowerPC] Change target data layout for 16-byte stack alignment (authored by saghir). Changed prior to commit:

[PATCH] D96265: [PowerPC] Change target data layout for 16-byte stack alignment

2021-02-12 Thread Nemanja Ivanovic via Phabricator via cfe-commits
nemanjai accepted this revision. nemanjai added a comment. This revision is now accepted and ready to land. Thank you for your patience. LGTM now. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D96265/new/ https://reviews.llvm.org/D96265

[PATCH] D96265: [PowerPC] Change target data layout for 16-byte stack alignment

2021-02-12 Thread Ahsan Saghir via Phabricator via cfe-commits
saghir updated this revision to Diff 323451. saghir added a comment. Updated tests to add run line for llc and target datalayout. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D96265/new/ https://reviews.llvm.org/D96265 Files:

[PATCH] D96265: [PowerPC] Change target data layout for 16-byte stack alignment

2021-02-12 Thread Ahsan Saghir via Phabricator via cfe-commits
saghir updated this revision to Diff 323307. saghir added a comment. Merged tests into one file. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D96265/new/ https://reviews.llvm.org/D96265 Files: clang/lib/Basic/Targets/PPC.h

[PATCH] D96265: [PowerPC] Change target data layout for 16-byte stack alignment

2021-02-11 Thread Nemanja Ivanovic via Phabricator via cfe-commits
nemanjai requested changes to this revision. nemanjai added a comment. This revision now requires changes to proceed. Can you please merge the tests into one file. There is no compelling reason to split them up and it is more difficult to review and make sense of what is going on. The test case

[PATCH] D96265: [PowerPC] Change target data layout for 16-byte stack alignment

2021-02-08 Thread Ahsan Saghir via Phabricator via cfe-commits
saghir created this revision. Herald added subscribers: shchenz, kbarton, hiraditya, nemanjai, emaste. saghir requested review of this revision. Herald added subscribers: llvm-commits, cfe-commits, MaskRay. Herald added projects: clang, LLVM. This changes the target data layout to make stack