atrosinenko wrote: Ping.
I have rebased this patch so that it is now on top of the stack of * #145379 - fix for MachineLICM which is needed for LOADgotAUTH to be dead-code-eliminated * #146488 and #146489 - the patches preventing immediate modifier substitution for sign and for auth/resign operations, respectively. Applying them before this patch instead of after it removes switching on the opcodes of `PAC[DI][AB](Z)?` instructions (which would be dropped later anyway): now the key (as per `AArch64PACKey::ID`) is taken from the operand of `AArch64::PAC` pseudo and the case of zero discriminator is not special anymore. Furthermore, this moves the discussion of `fixupBlendComponents` method to #146488, leaving this PR solely for the discussion of merging address materialization with the subsequent sign operation. Unfortunately, this PR was initially opened from my fork of llvm-project, so it is impossible to formally add it to the Graphite stack, thus I manually re-targeted it from `main` branch to `users/atrosinenko/pauth-imm-modifier-auth-resign`. Compared to #133788 this patch has slightly better coverage (supposedly due to cross-BB analysis) and is unified for the two ISel variants. This PR is inherently AArch64-specific, but we don't have other users of PtrAuth so far. Furthermore, implementing the same fix for GlobalISel in addition to DAGISel may require implementing something similar to this patch anyway, I guess. https://github.com/llvm/llvm-project/pull/130809 _______________________________________________ llvm-branch-commits mailing list llvm-branch-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
