[PATCH] D122929: [BOLT] Check for !isTailCall in isUnconditionalBranch

[Vladislav Khmelevsky via Phabricator via llvm-commits]

yota9 marked an inline comment as done.
yota9 added inline comments.


================
Comment at: bolt/test/AArch64/ext-double-jump.s:5
+# RUN: llvm-mc -filetype=obj -triple aarch64-unknown-unknown %s -o %t.o
+# RUN: %clang %cflags -nostartfiles -nodefaultlibs %t.o -o %t.exe -Wl,-q
+# RUN: llvm-bolt %t.exe -o %t.bolt -peepholes=double-jumps
----------------
Amir wrote:
> We already have `-nostartfiles` and `-nostdlib` in `%cflags` (bolt/test/AArch64/lit.local.cfg). Do we still need `-nostartfiles -nodefaultlibs` here?
Basically no, but from the stand-alone test point I would like them to be in such tests, where we don't need them by design


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D122929