atrosinenko wrote: @kovdan01 Thank you for the review, I have just pushed the commit that should resolve the comments.
Additionally, I discovered that I misunderstood BRA pseudo to be kind of tail call instruction - it is actually used for computed goto. While my original changes are still harmless, it is not so natural to declare BRA to clobber both X16 and X17 as it is not actually a call instruction, so I adjusted the definition of BRA and `emitPtrauthBranch` function a bit. https://github.com/llvm/llvm-project/pull/115185 _______________________________________________ llvm-branch-commits mailing list llvm-branch-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
