snehasish added a comment.
Thanks for the review.
================
Comment at: clang/include/clang/Driver/Options.td:2000
+defm split_machine_functions: OptInFFlag<"split-machine-functions",
+ "Enable", "Disable", " late function splitting using profile information
(x86-elf only)">;
+
----------------
MaskRay wrote:
> MaskRay wrote:
> > If this can be adapted to other targets, there is no need to specifically
> > mention x86-elf only ("x86 ELF")
> So I take quickly a look at rG94faadaca4e1704f674d2e9d4a1d25643b9ca52c. There
> is nothing ELF specific, right? At least I don't expect AArch64 ELF to fail.
> If you do think this merits a "ELF only" comment, adding it seems feasible.
The machine function splitter uses the basic block sections feature. This has
not been tested on AArch64, there is some target specific code in the CFI
handling as well idiosyncrasies of each target which need to be worked out. I'm
going to leave this as it is since the checks included in this patch enforce
x86 & ELF. This is on our longer term roadmap and we will revisit this in the
future.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D87047/new/
https://reviews.llvm.org/D87047
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
