peter.smith accepted this revision.
peter.smith added a comment.
This revision is now accepted and ready to land.

With moving the signature before the function entry this looks good to me. I'm 
not so familiar with the code in https://reviews.llvm.org/D148665 would ideally 
find someone a bit more experienced to take a look at that part. If I'm the 
only one available I can take a look but may take a bit more time.

We're looking to clarify when a BTI is required in the ABI 
https://github.com/ARM-software/abi-aa/issues/196 starting with some design 
options.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D148573

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to