[PATCH] D126177: [BOLT] [AArch64] Handle data markers spanning multiple functions

[Thorsten via Phabricator via llvm-commits]

tschuett added inline comments.


================
Comment at: bolt/lib/Core/BinaryContext.cpp:1646
+  // end of a data chunk inside code, $d indentifies start of data.
+  if (!isAArch64() || ELFSymbolRef(Symbol).getSize())
+    return MarkerSymType::NONE;
----------------
do you want to make the AARCH64 check an assert?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D126177