[PATCH] D128474: [BOLT] Support multiple parents for split jump table

[Huan Nguyen via Phabricator via llvm-commits]

nhuhuan added inline comments.


================
Comment at: bolt/lib/Core/BinaryContext.cpp:729
            << " due to cold fragments\n";
-  FragmentsToSkip.clear();
 }
----------------
There are two main types for FragmentsToSkip: (a) split jump table targets, (b) split landing pad targets.
Note that with our current code, we would need to call skipMarkedFragments() twice for (a) and (b). This
removal is to prepare for the next update where we only invoke skipMarkedFragments() once. To do that
we need to retain all FragmentsToSkip of both types.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D128474