[BOLT][NFC] Simplify BinaryFunction::setTrapOnEntry

[maksfb]

LGTM

bolt/include/bolt/Core/BinaryFunction.h

public:
   unsigned getLSDATypeEncoding() const { return LSDATypeEncoding; }
 
   const LSDATypeTableTy &getLSDATypeAddressTable() const {
     return LSDATypeAddressTable;
   }
 
   ArrayRef<uint8_t> getLSDATypeIndexTable() const { return LSDATypeIndexTable; }
 
-  const LabelsMapType &getLabels() const { return Labels; }
-
   IslandInfo &getIslandInfo() {
     assert(Islands && "function expected to have constant islands");
     return *Islands;
   }
 
   const IslandInfo &getIslandInfo() const {
     assert(Islands && "function expected to have constant islands");
     return *Islands;

bolt/lib/Core/BinaryFunction.cpp

for (MCSymbol *const EndLabel : FunctionEndLabels)
       if (EndLabel)
         BC.UndefinedSymbols.insert(EndLabel);
   }
 }
 
 void BinaryFunction::setTrapOnEntry() {
   clearDisasmState();
 
-  auto addTrapAtOffset = [&](uint64_t Offset) {
+  forEachEntryPoint([&](uint64_t Offset, const MCSymbol *Label) -> bool {
     MCInst TrapInstr;
     BC.MIB->createTrap(TrapInstr);
     addInstruction(Offset, std::move(TrapInstr));
-  };
-
-  addTrapAtOffset(0);
-  for (const std::pair<const uint32_t, MCSymbol *> &KV : getLabels())
-    if (getSecondaryEntryPointSymbol(KV.second))
-      addTrapAtOffset(KV.first);
+    return true;
+  });
 
   TrapsOnEntry = true;
 }
 
 void BinaryFunction::setIgnored() {
   if (opts::processAllFunctions()) {
     // We can accept ignored functions before they've been disassembled.
     // In that case, they would still get disassembled and emited, but not