[PATCH] D157157: [clang] Add clang support for Machine Function Splitting on AArch64

2023-09-06 Thread Daniel Hoekwater via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG776e3b052738: [clang] Add clang support for Machine Function Splitting on AArch64 (authored by dhoekwater). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D157

[PATCH] D157157: [clang] Add clang support for Machine Function Splitting on AArch64

2023-08-28 Thread Daniel Hoekwater via Phabricator via cfe-commits
dhoekwater updated this revision to Diff 554100. dhoekwater edited the summary of this revision. dhoekwater added a comment. Remove the dependency again Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D157157/new/ https://reviews.llvm.org/D157157 Fil

[PATCH] D157157: [clang] Add clang support for Machine Function Splitting on AArch64

2023-08-28 Thread Daniel Hoekwater via Phabricator via cfe-commits
dhoekwater updated this revision to Diff 554099. dhoekwater edited the summary of this revision. dhoekwater added a comment. Add the old dependency to hopefully get the change to propagate when I remove it Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.or

[PATCH] D157157: [clang] Add clang support for Machine Function Splitting on AArch64

2023-08-28 Thread Daniel Hoekwater via Phabricator via cfe-commits
dhoekwater updated this revision to Diff 554098. dhoekwater edited the summary of this revision. dhoekwater added a comment. Remove a dependency to see if that fixes things. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D157157/new/ https://reviews.

[PATCH] D157157: [clang] Add clang support for Machine Function Splitting on AArch64

2023-08-28 Thread Daniel Hoekwater via Phabricator via cfe-commits
dhoekwater updated this revision to Diff 554094. dhoekwater added a comment. Hopefully get rid of a dead dependency Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D157157/new/ https://reviews.llvm.org/D157157 Files: clang/lib/Driver/ToolChains/Cla

[PATCH] D157157: [clang] Add clang support for Machine Function Splitting on AArch64

2023-08-28 Thread Daniel Hoekwater via Phabricator via cfe-commits
dhoekwater updated this revision to Diff 554082. dhoekwater added a comment. Rebase onto main Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D157157/new/ https://reviews.llvm.org/D157157 Files: clang/lib/Driver/ToolChains/Clang.cpp clang/test/Dr

[PATCH] D157157: [clang] Add clang support for Machine Function Splitting on AArch64

2023-08-28 Thread Daniel Hoekwater via Phabricator via cfe-commits
dhoekwater added inline comments. Comment at: clang/lib/Driver/ToolChains/Clang.cpp:5852 options::OPT_fno_split_machine_functions)) { -// This codegen pass is only available on x86-elf targets. -if (Triple.isX86() && Triple.isOSBinFormatELF

[PATCH] D157157: [clang] Add clang support for Machine Function Splitting on AArch64

2023-08-28 Thread Daniel Hoekwater via Phabricator via cfe-commits
dhoekwater updated this revision to Diff 554070. dhoekwater marked 4 inline comments as done. dhoekwater edited the summary of this revision. dhoekwater added a comment. Make a comment more accurate and simplify test command functions Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACT

[PATCH] D157565: [CodeGen] Add AArch64 behavior to existing MFS tests

2023-08-22 Thread Daniel Hoekwater via Phabricator via cfe-commits
dhoekwater marked an inline comment as done. dhoekwater added a comment. Relanded in 90ab85a1b2e72f63039fadf6669b23f52192defd Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D157565

[PATCH] D157750: Properly handle -fsplit-machine-functions for fatbinary compilation

2023-08-16 Thread Daniel Hoekwater via Phabricator via cfe-commits
dhoekwater added a comment. This patch will make it difficult to write tests for MFS on AArch64 before it is officially enabled. Currently, because clang performs the Triple check, we can use `-enable-split-machine-functions` to run tests with MFS on Arm, but after this patch the flag won't do

[PATCH] D157750: Properly handle -fsplit-machine-functions for fatbinary compilation

2023-08-16 Thread Daniel Hoekwater via Phabricator via cfe-commits
dhoekwater accepted this revision. dhoekwater added a comment. This revision is now accepted and ready to land. lgtm. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D157750/new/ https://reviews.llvm.org/D157750 __

[PATCH] D157157: [clang] Add clang support for Machine Function Splitting on AArch64

2023-08-07 Thread Daniel Hoekwater via Phabricator via cfe-commits
dhoekwater updated this revision to Diff 547884. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D157157/new/ https://reviews.llvm.org/D157157 Files: clang/lib/Driver/ToolChains/Clang.cpp clang/test/Driver/fsplit-machine-functions.c Index: clang/

[PATCH] D157157: [clang] Add clang support for Machine Function Splitting on AArch64

2023-08-04 Thread Daniel Hoekwater via Phabricator via cfe-commits
dhoekwater created this revision. dhoekwater added reviewers: shenhan, xur, snehasish, wenlei, mingmingl. Herald added subscribers: pengfei, kristof.beyls. Herald added a project: All. dhoekwater requested review of this revision. Herald added subscribers: cfe-commits, MaskRay. Herald added a proje