[PATCH] D127111: [BOLT][AArch64] Handle data at the beginning of a function when disassembling and building CFG.

Mon Jun 6 08:25:35 PDT 2022

treapster added inline comments.

================
Comment at: bolt/test/AArch64/data-at-0-offset.c:3
+// RUN: llvm-bolt %t.exe -o %t.bolt 2>&1 | FileCheck %s
+// CHECK-NOT: BOLT-WARNING
+
----------------
yota9 wrote:
> Which warning is supposed to be here? Please update with more specific string
Well, without my patch it just fails with assertion `llvm-project/bolt/lib/Core/BinaryFunction.cpp:1982: bool llvm::bolt::BinaryFunction::buildCFG(llvm::bolt::MCPlusBuilder::AllocatorIdTy): Assertion 'PrevBB && "no previous basic block for a fall through"' failed.` But just in case i now added check for disassembling failure.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D127111