[PATCH] D143522: [AMDGPU] Set a data layout entry for buffer descriptors (addrspace 7)

2023-02-16 Thread Nicolai Hähnle via Phabricator via cfe-commits
nhaehnle added a comment. This really needs to be a 160-bit type, not a 128-bit type. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D143522/new/ https://reviews.llvm.org/D143522 ___ cfe-commits mailing li

[PATCH] D143522: [AMDGPU] Set a data layout entry for buffer descriptors (addrspace 7)

2023-02-14 Thread Krzysztof Drewniak via Phabricator via cfe-commits
krzysz00 added a reviewer: piotr. krzysz00 added a comment. Discussion related to this is also happening over at https://reviews.llvm.org/D143945 . Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D143522/new/ https://reviews.llvm.org/D143522 ___

[PATCH] D143522: [AMDGPU] Set a data layout entry for buffer descriptors (addrspace 7)

2023-02-10 Thread Krzysztof Drewniak via Phabricator via cfe-commits
krzysz00 updated this revision to Diff 496515. krzysz00 added a comment. Herald added a project: clang. Herald added a subscriber: cfe-commits. Apparently clang has its own copy of this data layout. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D1435