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
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
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
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.
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
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
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
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
MaskRay 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())
mingmingl accepted this revision.
mingmingl added inline comments.
This revision is now accepted and ready to land.
Comment at: clang/test/Driver/fsplit-machine-functions.c:4
// RUN: %clang -### -target x86_64 -fprofile-use=default.profdata
-fsplit-machine-functions -fno-split-
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/
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
12 matches
Mail list logo