[PATCH] D104931: [AArch64] Wire up ILP32 ABI support in Clang

2023-07-17 Thread Fangrui Song via Phabricator via cfe-commits
MaskRay added a comment. Herald added a subscriber: wangpc. We need some tests. https://maskray.me/blog/2021-08-08-toolchain-testing#i-dont-know-whether-a-test-is-needed Adding some to `clang/test/Preprocessor/init-aarch64.c` should cover many changes, but we also need one to cover

[PATCH] D104931: [AArch64] Wire up ILP32 ABI support in Clang

2022-12-29 Thread Amanieu d'Antras via Phabricator via cfe-commits
Amanieu added a comment. The situation around -mabi for AArch64 is a bit awkward: Clang uses it to select the ARM64 calling convention (`-mabi=aapcs` vs `-mabi=darwinpcs`) while GCC uses it to select between LP64 and ILP32. I'm not sure how we should be handling this. Repository: rG LLVM

[PATCH] D104931: [AArch64] Wire up ILP32 ABI support in Clang

2022-12-22 Thread Nick Desaulniers via Phabricator via cfe-commits
nickdesaulniers added a comment. Herald added a project: All. The linux kernel uses a build time invocation of the compiler for feature detection of the `-mabi=lp64` command line flag, which clang-16 currently doesn't support. Looking at https://gcc.gnu.org/onlinedocs/gcc/AArch64-Options.html,

[PATCH] D104931: [AArch64] Wire up ILP32 ABI support in Clang

2022-02-22 Thread Amanieu d'Antras via Phabricator via cfe-commits
Amanieu added a comment. Ping. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D104931/new/ https://reviews.llvm.org/D104931 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D104931: [AArch64] Wire up ILP32 ABI support in Clang

2021-07-22 Thread Amanieu d'Antras via Phabricator via cfe-commits
Amanieu added a comment. Ping. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D104931/new/ https://reviews.llvm.org/D104931 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D104931: [AArch64] Wire up ILP32 ABI support in Clang

2021-06-25 Thread Amanieu d'Antras via Phabricator via cfe-commits
Amanieu created this revision. Amanieu added reviewers: t.p.northover, aemerson, kristof.beyls. Herald added a subscriber: danielkiss. Amanieu requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. This is a follow-up to D94143