[BOLT] LongJmp: Check for shouldEmit

[yota9]

Summary

Check that the function will be emitted in the final binary. Preserving
old function address is needed in case it is PLT trampoline, it is
currently not moved by the BOLT.

[rafauler]

LGTM

bolt/lib/Core/BinaryContext.cpp

for (size_t I = 0, Size = FileNames.size(); I != Size; ++I) {
       cantFail(getDwarfFile(Dir, FileName, 0, None, None, CUID));
     }
   }
 
   preprocessDWODebugInfo();
 }
 
 bool BinaryContext::shouldEmit(const BinaryFunction &Function) const {
+  if (Function.isPseudo())
+    return false;
+
   if (opts::processAllFunctions())
     return true;
 
   if (Function.isIgnored())
     return false;
 
   // In relocation mode we will emit non-simple functions with CFG.
   // If the function does not have a CFG it should be marked as ignored.

bolt/lib/Passes/LongJmp.cpp

for (BinaryFunction *BF : SortedFunctions) {
       ++CurrentIndex;
     }
   }
 
   // Hot
   CurrentIndex = 0;
   bool ColdLayoutDone = false;
   for (BinaryFunction *Func : SortedFunctions) {
+    if (!BC.shouldEmit(*Func)) {
+      HotAddresses[Func] = Func->getAddress();
+      continue;
+    }
+
     if (!ColdLayoutDone && CurrentIndex >= LastHotIndex) {
       DotAddress =
           tentativeLayoutRelocColdPart(BC, SortedFunctions, DotAddress);
       ColdLayoutDone = true;
       if (opts::HotFunctionsAtEnd)
         DotAddress = alignTo(DotAddress, opts::AlignText);
     }