[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 Apr 9 16:55:13 PDT 2021


MaskRay accepted this revision.
MaskRay added a comment.

LGTM.



================
Comment at: llvm/test/CodeGen/AArch64/patchable-function-entry-bti.ll:51
 ;; -fpatchable-function-entry=1 -mbranch-protection=bti
 ;; For M=0, don't create .Lpatch0 if the initial instruction is not BTI,
 ;; even if other basic blocks may have BTI.
----------------
The comment needs update. It should say that we add BTI c even if the function has the internal linkage.


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