[PATCH] D130521: [BOLT] Generate sections for multiple fragments

Rafael Auler via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 17 17:22:55 PDT 2022


rafauler accepted this revision.
rafauler added a comment.
This revision is now accepted and ready to land.

LGTM



================
Comment at: bolt/lib/Core/BinaryFunction.cpp:4062
+        const MCSymbol *ColdStartSymbol = getSymbol(FF.getFragmentNum());
+        // If fragment is empty, symbol might not not emitted
+        if (FF.empty() && (!ColdStartSymbol || !ColdStartSymbol->isDefined()) &&
----------------
symbol might have not been emitted


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D130521



More information about the llvm-commits mailing list