[PATCH] D140457: [BOLT] Process fragment siblings in lite mode, keep lite mode on

Maksim Panchenko via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Jan 8 18:48:11 PST 2023


maksfb added a comment.

If the function is local, it will have a `/<N>` suffix inserted after the main fragment name and after `.cold`. Will this change cover such case? Please add a test for it.



================
Comment at: bolt/lib/Rewrite/RewriteInstance.cpp:1460-1461
     if (FragName) {
+      if (!BC->HasSplitFunctions)
+        BC->HasSplitFunctions = true;
       static bool PrintedWarning = false;
----------------



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D140457



More information about the llvm-commits mailing list