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