[BOLT] Postprocess entry points after disassembly

[nhuhuan]

Summary

When processing split jump tables and split landing pad, BOLT generates
additional secondary entry points, which all then should go through the
postprocessing checks.

In this update, postProcessEntryPoints is performed after disassembly
and branch analysis are completed.

Test Plan:

ninja check-bolt

bolt/lib/Rewrite/RewriteInstance.cpp

if (opts::PrintAll || opts::PrintDisasm)
       Function.print(outs(), "after disassembly", true);
   }
 
   BC->processInterproceduralReferences();
   BC->populateJumpTables();
 
   for (auto &BFI : BC->getBinaryFunctions()) {
     BinaryFunction &Function = BFI.second;
-
     if (!shouldDisassemble(Function))
       continue;
-
-    Function.postProcessEntryPoints();
     Function.postProcessJumpTables();
   }
 
   BC->clearJumpTableTempData();
   BC->adjustCodePadding();
 
   for (auto &BFI : BC->getBinaryFunctions()) {
     BinaryFunction &Function = BFI.second;
if (!Function.trapsOnEntry() && !CFIRdWrt->fillCFIInfoFor(Function)) {
       }
     }
 
     // Parse LSDA.
     if (Function.getLSDAAddress() != 0 &&
         !BC->getFragmentsToSkip().count(&Function))
       Function.parseLSDA(getLSDAData(), getLSDAAddress());
   }
+
+  // Split jump table, split landing pad register secondary entry points
+  // postProcessEntryPoints needs to be invoked in the end
+  for (auto &BFI : BC->getBinaryFunctions()) {
+    BinaryFunction &Function = BFI.second;
+    if (!shouldDisassemble(Function))
+      continue;
+    Function.postProcessEntryPoints();
+  }
 }
 
 void RewriteInstance::buildFunctionsCFG() {
   NamedRegionTimer T("buildCFG", "buildCFG", "buildfuncs",
                      "Build Binary Functions", opts::TimeBuild);
 
   // Create annotation indices to allow lock-free execution
   BC->MIB->getOrCreateAnnotationIndex("JTIndexReg");