[PATCH] D108792: [M68k] Update pointer data layout

2021-08-27 Thread Ricky Taylor 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 rG8d3f112f0cdb: [M68k] Update pointer data layout (authored by ricky26). Changed prior to commit:

[PATCH] D108792: [M68k] Update pointer data layout

2021-08-27 Thread John Paul Adrian Glaubitz via Phabricator via cfe-commits
glaubitz added a comment. @ricky26 Could you also get this fix backported in the LLVM fork of Rust? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D108792/new/ https://reviews.llvm.org/D108792 ___

[PATCH] D108792: [M68k] Update pointer data layout

2021-08-26 Thread Jessica Clarke via Phabricator via cfe-commits
jrtc27 accepted this revision. jrtc27 added a comment. This revision is now accepted and ready to land. Comments could perhaps be clearer about highlighting that 32-bit integers/pointers are only 16-bit aligned in the ABI as that's the weird bit, not the 32-bit preferred alignment Repository:

[PATCH] D108792: [M68k] Update pointer data layout

2021-08-26 Thread Ricky Taylor via Phabricator via cfe-commits
ricky26 created this revision. ricky26 added reviewers: myhsu, jrtc27, nickdesaulniers, craig.topper. Herald added a subscriber: hiraditya. ricky26 requested review of this revision. Herald added projects: clang, LLVM. Fixes PR51626. The M68k requires that all instruction, word and long word