[PATCH] D99417: [AArch64][v8.5A] Add BTI to all function starts

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 26 13:48:32 PDT 2021


MaskRay added a comment.

I know that for generalization, sometimes specific tests can be removed. -fpatchable-function-entry is a bit special though, it can place NOPs before and after the function label. This property is relied upon by the Linux kernel dynamic ftrace.
Since the NOP insertion is a bit strange, and testing it does not increase test time (we don't add new `llc` commands so no process startup overhead), I'd like patchable-function-entry-bti.ll to be kept.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99417



More information about the llvm-commits mailing list