================ @@ -717,6 +716,466 @@ SrcSafetyAnalysis::create(BinaryFunction &BF, RegsToTrackInstsFor); } +/// A state representing which registers are safe to be used as the destination +/// operand of an authentication instruction. +/// +/// Similar to SrcState, it is the analysis that should take register aliasing +/// into account. +/// ---------------- kbeyls wrote:
At first sight, it looks like there is quite a bit of code added here for `DstState` and `DstSafetyAnalysis`. I wonder if you have thought about/analyzed if it would be (a) possible and (b) desirable to try and share more of the implementation between the `Dst...` and `Src...` classes? I haven't investigated this at all --- just thought I'd ask you first in case you had thought about this before. https://github.com/llvm/llvm-project/pull/135663 _______________________________________________ llvm-branch-commits mailing list llvm-branch-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits