[PATCH] D88041: [lld] Add a flag to enable split machine functions for LTO.

Teresa Johnson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 22 08:26:28 PDT 2020


tejohnson added a comment.

Is it expected that the user will want to specify this only at link time and not during compile time? If it is normally specified in the compile step, you should consider adding it to the IR in some fashion (e.g. function attributes). The advantage is that the user doesn't need to pass different options for the LTO and non-LTO cases.



================
Comment at: lld/test/ELF/lto/split-machine-functions.ll:10
+define void @foo1(i1 zeroext %0) nounwind !prof !14 !section_prefix !15 {
+;; Check that cold block is split out with new symbol. 
+; CHECK:  foo1.cold
----------------
Check that the opposite occurs by default and with the "no" variant


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D88041/new/

https://reviews.llvm.org/D88041



More information about the llvm-commits mailing list